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
[feat] console - add pagination #254
Conversation
The "last page" button is disabled for now, as the registry does not sends back the total number of devices when limiting. So we cannot calculate the total number of page. Allow to pass query options to the wasm request builder
/test |
Aye, aye, captain! … Triggered System Test worklow! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Let's check what the CI has to say about it.
I am not happy with the added "query" parameter. However, I am also not happy with the other parameters of this function 😁 … so let's keep this pattern for now (as it is consistent), but maybe refactor this into some more builder-style pattern in the future.
Git:
Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/2345953449 Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/05/18/test-run-2345953449.html |
The "last page" button is acting as a "next" button, as the registry does not
sends back the total number of devices when limiting, so we cannot calculate the total number of pages. This seems to be the recommended behaviour in patternfly examples.
Allow to pass query options to the wasm request builder