Skip to content

refactor: [M3-8127] - Query Key Factory for Domains#10559

Merged
bnussman-akamai merged 2 commits intolinode:developfrom
bnussman-akamai:M3-8127-query-key-factory-domains
Jun 11, 2024
Merged

refactor: [M3-8127] - Query Key Factory for Domains#10559
bnussman-akamai merged 2 commits intolinode:developfrom
bnussman-akamai:M3-8127-query-key-factory-domains

Conversation

@bnussman-akamai
Copy link
Member

@bnussman-akamai bnussman-akamai commented Jun 10, 2024

Description 📝

  • Updates domain queries to use the query key factory 🏭
  • Updates functions to use single object notation to prepare for the v4 to v5 upgrade (see here) 🔧

Preview 📷

Note

No UI changes expected

How to test 🧪

  • Test general functionality of Domains in Cloud Manager 👀🌎

As an Author I have considered 🤔

  • 👀 Doing a self review
  • ❔ Our contribution guidelines
  • 🤏 Splitting feature into small PRs
  • ➕ 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

@bnussman-akamai bnussman-akamai added the React Query Relating to the transition to use React Query label Jun 10, 2024
@bnussman-akamai bnussman-akamai self-assigned this Jun 10, 2024
@bnussman-akamai bnussman-akamai requested a review from a team as a code owner June 10, 2024 16:07
@bnussman-akamai bnussman-akamai requested review from abailly-akamai and hkhalil-akamai and removed request for a team June 10, 2024 16:07
Copy link
Contributor

@hkhalil-akamai hkhalil-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Tested creating, viewing, editing and deleting domains and verified changes are reflected in a parallel browser window.

Copy link
Contributor

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good and noticed no regression while testing adding, editing, deleting domains ✅

@github-actions
Copy link

Coverage Report:
Base Coverage: 82.29%
Current Coverage: 82.29%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

React Query Relating to the transition to use React Query

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants