Skip to content

CLOUDP-332090/proper-pagination #119

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

Merged
merged 13 commits into from
Jul 28, 2025
Merged

Conversation

andrpac
Copy link
Collaborator

@andrpac andrpac commented Jul 23, 2025

Proposed changes

This represents a proper pagination fix that loads all of the pages with the store, unlike the previous version which only returned one page.

Jira ticket: CLOUDP-332090

Checklist

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added any necessary documentation in the document requirements section listed in CONTRIBUTING.md (if appropriate)
  • I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
  • I have run make fmt and formatted my code

Further comments

Note as of now the fix is only for ipAcessList as a sample of what will happen with all of them.

@andrpac andrpac requested a review from a team as a code owner July 23, 2025 10:17
@andrpac andrpac force-pushed the CLOUDP-332090/proper-pagination branch 3 times, most recently from 7f9bfb1 to e30d613 Compare July 23, 2025 12:45
@andrpac andrpac force-pushed the CLOUDP-332090/proper-pagination branch 7 times, most recently from 177b680 to 40ab4cc Compare July 24, 2025 11:45
@andrpac
Copy link
Collaborator Author

andrpac commented Jul 24, 2025

Q: Is there a way to test identity providers? Reference manifest does not include any information about them?

@andrpac andrpac force-pushed the CLOUDP-332090/proper-pagination branch 5 times, most recently from 42d1472 to b93b80b Compare July 25, 2025 12:49
@andrpac andrpac force-pushed the CLOUDP-332090/proper-pagination branch from b93b80b to f754a9e Compare July 25, 2025 14:12
Copy link
Collaborator

@josvazg josvazg left a comment

Choose a reason for hiding this comment

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

LGTM with a nitpick rename ask

@andrpac andrpac force-pushed the CLOUDP-332090/proper-pagination branch from f754a9e to a9e35b5 Compare July 28, 2025 08:39
@andrpac andrpac merged commit e5d48d2 into main Jul 28, 2025
15 checks passed
@andrpac andrpac deleted the CLOUDP-332090/proper-pagination branch July 28, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants