Skip to content
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

ref(integrations): Allow pagination on team and user mappings pages #29862

Merged
merged 3 commits into from
Nov 10, 2021

Conversation

leeandher
Copy link
Member

@leeandher leeandher commented Nov 9, 2021

See API 2204

This PR will allow the team mappings and user mappings pages to paginate when they exceed 100 entries. The pagination will be disabled if there are less than 100 entries.

It should be noted that there aren't necessarily only 100 entries in the UI when paginating. We paginate based on the sentry mapping, not the external one. For example: If #sentry in sentry maps to @sentry-fe and @sentry-be in github, both will appear on the page that renders the mappings to #sentry. This behavior can be seen in the mocked demo below (where the page limit was changed to 2 entries).

DEMO

Untitled.mov

TODO:

  • Test preventing deleted entries from taking up pagination space

@leeandher leeandher requested a review from a team as a code owner November 9, 2021 01:05
@leeandher leeandher requested a review from a team November 9, 2021 01:05
@leeandher leeandher changed the title Leander page map/api 2204 ref(integrations): Allow pagination on team and user mappings pages Nov 9, 2021
@leeandher leeandher merged commit 3f86a60 into master Nov 10, 2021
@leeandher leeandher deleted the leander-page-map/API-2204 branch November 10, 2021 21:11
@github-actions github-actions bot locked and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants