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

Table/API pagination does not respect MAX_PAGE_SIZE #7101

Closed
jeremystretch opened this issue Aug 31, 2021 · 1 comment
Closed

Table/API pagination does not respect MAX_PAGE_SIZE #7101

jeremystretch opened this issue Aug 31, 2021 · 1 comment
Assignees
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@jeremystretch
Copy link
Member

jeremystretch commented Aug 31, 2021

NetBox version

v3.0.0

Python version

3.7

Steps to Reproduce

  1. Create at least 100 IP addresses
  2. Set MAX_PAGE_SIZE = 10 in `configuration.py
  3. View the IP addresses list and set the page size to 50

Expected Behavior

Only the first 10 IP addresses should be displayed.

Observed Behavior

The first 50 IP addresses are displayed; MAX_PAGE_SIZE has no effect.

(FYI this bug exists in releases prior to v3.0 as well, but was discovered in the course of troubleshooting something unrelated.)

@jeremystretch jeremystretch added type: bug A confirmed report of unexpected behavior in the application status: accepted This issue has been accepted for implementation labels Aug 31, 2021
@jeremystretch jeremystretch self-assigned this Aug 31, 2021
@jeremystretch jeremystretch changed the title Table pagination does not respect MAX_PAGE_SIZE Table/API pagination does not respect MAX_PAGE_SIZE Aug 31, 2021
@jeremystretch
Copy link
Member Author

It seems that a similar issue exists with the REST API as well.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

No branches or pull requests

2 participants
@jeremystretch and others