-
Notifications
You must be signed in to change notification settings - Fork 20
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
Inventory list needs to reset offset when paging limit increased #400
Comments
Good catch @cdcabrera |
@ntkathole if you could also confirm this has been resolved before we close this one down, that would be awesome! |
@cdcabrera This is still an issue. We should also reset offset when new filter applied. Here are steps:
|
@ntkathole updates around reseting the paging when manipulating filters should be in CI-beta when you're available to confirm |
@cdcabrera There is still issue for openshift cores/socket filter. Here are steps:
|
Looks like we need a better way of handling this, investigating something a little bit more centralized so when we add additional filters we don't need to update multiple locations of paging resets. |
Third round for the win, I hope =) @ntkathole updates around reseting the paging when manipulating filters should be in CI-beta when you're available to confirm We'll be squashing the last 3 #400 bug fix commits down into a single commit after confirmation, and doing the same for the #383 commits so we can track them a bit easier |
@cdcabrera Issue is fixed now. Thank you 🎉 |
Description
Appears to be a bug, the offset/page doesn't reset when the paging limit is increased. this can lead to an API 4XX level error
Expected behavior
The GUI shouldn't break when increasing the paging limit.
Steps to reproduce
Additional context, version info
The text was updated successfully, but these errors were encountered: