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

Fix missing paging on models #all methods #4

Merged
merged 2 commits into from
Oct 25, 2023

Conversation

agrare
Copy link
Member

@agrare agrare commented Oct 25, 2023

Most of the models' #all methods are missing paging which limits the number of results to a maximum of 500.

If there are more instances than will fit in a page a next_page_token is
returned.  This has to be passed in to subsequent calls in order to
retrieve the full list of servers.
Most of the models' `#all` methods are missing paging which limits the
number of results to a maximum of 500.
@agrare agrare added the bug Something isn't working label Oct 25, 2023
@Fryguy Fryguy merged commit 81c0c2f into ManageIQ:1_22_0 Oct 25, 2023
agrare added a commit that referenced this pull request Oct 25, 2023
Fixed
- Fix missing paging on models' `#all` methods (#4)
- Use `Minitest::Test` instead of `MiniTest::Test` (#2)
@agrare agrare deleted the fix_all_paging branch October 30, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants