-
Notifications
You must be signed in to change notification settings - Fork 24
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
chore: Migrate useEnterpriseAccountDetails to TS Query V5 #3571
chore: Migrate useEnterpriseAccountDetails to TS Query V5 #3571
Conversation
54fc3c9
to
81f0690
Compare
Bundle ReportChanges will increase total bundle size by 140 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
Bundle ReportChanges will increase total bundle size by 140 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3571 +/- ##
=======================================
Coverage 98.90% 98.90%
=======================================
Files 806 806
Lines 14485 14485
Branches 4116 4109 -7
=======================================
Hits 14327 14327
Misses 151 151
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3571 +/- ##
=======================================
Coverage 98.90% 98.90%
=======================================
Files 806 806
Lines 14485 14485
Branches 4109 4109
=======================================
Hits 14327 14327
Misses 151 151
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #3571 +/- ##
=======================================
Coverage 98.90% 98.90%
=======================================
Files 806 806
Lines 14485 14485
Branches 4109 4116 +7
=======================================
Hits 14327 14327
Misses 151 151
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #3571 +/- ##
=======================================
Coverage 98.90% 98.90%
=======================================
Files 806 806
Lines 14485 14485
Branches 4109 4109
=======================================
Hits 14327 14327
Misses 151 151
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
src/pages/PlanPage/subRoutes/CurrentOrgPlan/queries/EnterpriseAccountDetailsQueryOpts.ts
Outdated
Show resolved
Hide resolved
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One little thing, lgtm otherwise
Description
This PR migrates the
useEnterpriseAccountDetails
to the queryOptions API versionEnterpriseAccountDetailsQueryOpts
Ticket: codecov/engineering-team#2968
Notable Changes
useEnterpriseAccountDetails
toEnterpriseAccountDetailsQueryOpts
CurrentOrgPlan
andAccountOrgs