Skip to content
New issue

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

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

Already on GitHub? # to your account

icu-14735 add search & pagination to aliases #2498

Merged

Conversation

lisbet-alvarez
Copy link
Collaborator

✅ 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

  1. delete current indexedDB since alias index was added.
  2. go to "Aliases"
  3. experiment with searching & switching between pages.

Checklist

  • I have added before and after screenshots for UI changes
  • I have added JSON response output for API changes
  • I have added steps to reproduce and test for bug fixes in the description
  • I have commented on my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Copy link

vercel bot commented Sep 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
boundary-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 30, 2024 7:38pm
boundary-ui-desktop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 30, 2024 7:38pm

@DhariniJeeva
Copy link
Collaborator

Thanks for introducing pagination to Aliases 👏
I added a couple of comments, looks great otherwise!

Co-authored-by: Dharini Jeeva <dharini@hashicorp.com>
Copy link
Collaborator

@DhariniJeeva DhariniJeeva left a 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 <cameron.perera@hashicorp.com>
Copy link
Collaborator

@cameronperera cameronperera left a comment

Choose a reason for hiding this comment

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

LGTM

@cameronperera cameronperera added this to the 0.18.0 milestone Sep 30, 2024
Copy link
Collaborator

@ZedLi ZedLi left a 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

@lisbet-alvarez lisbet-alvarez merged commit 91ffbfb into main Sep 30, 2024
12 checks passed
@lisbet-alvarez lisbet-alvarez deleted the ICU-14735-boundary-pagination-for-the-aliases-page-in-the-ui branch September 30, 2024 20:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants