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

Increase default per-page API results limit #246

Merged
merged 1 commit into from
May 25, 2024

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented May 25, 2024

Increase the per-page results limit from 20 results per query to 50 and allow overriding this value via the new CLI flag.

Planned future work will add pagination support to properly handle paging results at which point we can reduce our this application default value back to 20 results to match the API default paging limit.

refs GH-245

Increase the per-page results limit from 20 results per query
to 50 and allow overriding this value via the new CLI flag.

Planned future work will add pagination support to properly
handle paging results at which point we can reduce our this
application default value back to 20 results to match the API
default paging limit.

refs GH-245
@atc0005 atc0005 added bug Something isn't working documentation Improvements or additions to documentation plugin/sync plans app/lssp List (ls) Satellite Sync Plans (sp) labels May 25, 2024
@atc0005 atc0005 added this to the Next Dev Release milestone May 25, 2024
@atc0005 atc0005 self-assigned this May 25, 2024
@atc0005 atc0005 merged commit f77dfb8 into development May 25, 2024
29 checks passed
@atc0005 atc0005 deleted the i245-increase-default-api-per-page-limit branch May 25, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app/lssp List (ls) Satellite Sync Plans (sp) bug Something isn't working documentation Improvements or additions to documentation plugin/sync plans
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant