-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: page number reset #8681
fix: page number reset #8681
Conversation
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
2 similar comments
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
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!
@Tooluloope The build has failed so the checks cannot progress. The build has failed due to a jest test failing. Please check the details link in the Appsmith Client Build Workflow. |
Not sure why, just re-ran and it passed |
@Tooluloope Tested this PR and it is working as expected now |
@Tooluloope merge latest release, Run ok to test and make sure to merge this as soon as everything is done |
/ok-to-test sha=8eba3b4 |
1 similar comment
/ok-to-test sha=8eba3b4 |
Description
Table widget - server-side pagination control: Page number is reset to 1 once the query has completed
Fixes #8621
Type of change
Checklist:
Test coverage results π§ͺ
βͺ Total coverage has not changed