-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
fix: Pagination not working on Client Keys page #99562
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
base: master
Are you sure you want to change the base?
fix: Pagination not working on Client Keys page #99562
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #99562 +/- ##
==========================================
- Coverage 81.24% 81.24% -0.01%
==========================================
Files 8588 8588
Lines 380057 380056 -1
Branches 24074 24074
==========================================
- Hits 308771 308770 -1
Misses 70923 70923
Partials 363 363 |
} = useApiQuery<ProjectKey[]>( | ||
[ | ||
`/projects/${organization.slug}/${projectId}/keys/`, | ||
{ | ||
query: { | ||
cursor: decodeScalar(location.query.cursor), | ||
}, | ||
}, | ||
], | ||
{ | ||
staleTime: 0, | ||
} | ||
); |
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.
Should we pass in the per-page attribute here as well? I'm aware of #99563, but if that default changes at any point in time, this page will look different again.
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.
yep, that would be even better solution. We can change the default on backend never the less, but ideally this info should be coming from the client/frontend
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.
LGTM 🚀
closes https://linear.app/getsentry/issue/TET-1135/project-settigns-client-keys-pagination-does-not-work