Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: [M3-7444] - Query Key Factory for Status Page #10672

Merged
merged 5 commits into from
Jul 17, 2024

Conversation

jaalah-akamai
Copy link
Contributor

Description πŸ“

  • Updates StatusPage to use a Query Key factory 🏭

Target release date πŸ—“οΈ

7/22

How to test πŸ§ͺ

  • Verify Cypress tests pass βœ…
  • Verify unit tests pass βœ…
  • Manually change scheduled to completed and see banner still works

As an Author I have considered πŸ€”

Check all that apply

  • πŸ‘€ Doing a self review
  • ❔ Our contribution guidelines
  • 🀏 Splitting feature into small PRs
  • [x ] βž• Adding a changeset
  • [] πŸ§ͺ Providing/Improving test coverage
  • πŸ” Removing all sensitive information from the code and PR description
  • 🚩 Using a feature flag to protect the release
  • πŸ‘£ Providing comprehensive reproduction steps
  • πŸ“‘ Providing or updating our documentation
  • πŸ•› Scheduling a pair reviewing session
  • πŸ“± Providing mobile support
  • β™Ώ Providing accessibility support

@jaalah-akamai jaalah-akamai added React Query Relating to the transition to use React Query Ready for Review labels Jul 11, 2024
@jaalah-akamai jaalah-akamai self-assigned this Jul 11, 2024
@jaalah-akamai jaalah-akamai requested a review from a team as a code owner July 11, 2024 14:48
@jaalah-akamai jaalah-akamai requested review from bnussman-akamai and AzureLatte and removed request for a team July 11, 2024 14:48
Copy link

github-actions bot commented Jul 11, 2024

Coverage Report: βœ…
Base Coverage: 82.48%
Current Coverage: 82.48%

@jaalah-akamai jaalah-akamai added the Add'tl Approval Needed Waiting on another approval! label Jul 15, 2024
@jaalah-akamai jaalah-akamai merged commit 0822ae6 into linode:develop Jul 17, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Add'tl Approval Needed Waiting on another approval! React Query Relating to the transition to use React Query Ready for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants