-
Notifications
You must be signed in to change notification settings - Fork 29
icu-14735 add search & pagination to aliases #2498
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
icu-14735 add search & pagination to aliases #2498
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…he-ui' of https://github.com/hashicorp/boundary-ui into ICU-14735-boundary-pagination-for-the-aliases-page-in-the-ui
Thanks for introducing pagination to Aliases 👏 |
Co-authored-by: Dharini Jeeva <[email protected]>
…he-ui' of https://github.com/hashicorp/boundary-ui into ICU-14735-boundary-pagination-for-the-aliases-page-in-the-ui
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.
looks great!
Co-authored-by: Cameron Perera <[email protected]>
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
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.
Looks good! 1M aliases is incredibly slow but at least it loads
✅ Closes: https://hashicorp.atlassian.net/browse/ICU-14735
Description
Add search & pagination to aliases.
NOTE: We will need to update db version so it doesn't break for users. (There is a draft pr to do this so we might want to revisit that PR and update it if neccessary).
There will still be some upcoming changes to ensure search & pagination does not break when there are 500k+ aliases.
Screenshots (if appropriate)
Screen.Recording.2024-09-23.at.5.44.45.PM.mov
How to Test
Checklist