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
[Security Solution][Exceptions] - Fix pagination bug on shared exception lists view #151393
Conversation
…nging number of rows displayed after page 1
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
const onPerPageClick = useCallback((size: number) => { | ||
closeRowSizePopover(); | ||
setRowSize(size); | ||
setActivePage(0); |
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.
This is where the fix is. There's some jank with the page, reloading when I don't think we need to on row selection, but for now just fixing this bug.
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @yctercero |
…ion lists view (elastic#151393) ## Summary Fixes elastic#147849 When changing number of rows in exception lists view from any page other than page 1. (cherry picked from commit 250dcff)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…exception lists view (#151393) (#151927) # Backport This will backport the following commits from `main` to `8.7`: - [[Security Solution][Exceptions] - Fix pagination bug on shared exception lists view (#151393)](#151393) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Yara Tercero","email":"yctercero@users.noreply.github.com"},"sourceCommit":{"committedDate":"2023-02-22T18:55:11Z","message":"[Security Solution][Exceptions] - Fix pagination bug on shared exception lists view (#151393)\n\n## Summary\r\n\r\nFixes #147849 changing number of rows in exception lists view from any page other\r\nthan page 1.","sha":"250dcffcdab4a63411713d638185b0c1c7c84687","branchLabelMapping":{"^v8.8.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:fix","Team:Detections and Resp","Feature:Rule Exceptions","Team:Security Solution Platform","v8.7.0","v8.8.0"],"number":151393,"url":"#151393 Solution][Exceptions] - Fix pagination bug on shared exception lists view (#151393)\n\n## Summary\r\n\r\nFixes #147849 changing number of rows in exception lists view from any page other\r\nthan page 1.","sha":"250dcffcdab4a63411713d638185b0c1c7c84687"}},"sourceBranch":"main","suggestedTargetBranches":["8.7"],"targetPullRequestStates":[{"branch":"8.7","label":"v8.7.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.8.0","labelRegex":"^v8.8.0$","isSourceBranch":true,"state":"MERGED","url":"#151393 Solution][Exceptions] - Fix pagination bug on shared exception lists view (#151393)\n\n## Summary\r\n\r\nFixes #147849 changing number of rows in exception lists view from any page other\r\nthan page 1.","sha":"250dcffcdab4a63411713d638185b0c1c7c84687"}}]}] BACKPORT--> Co-authored-by: Yara Tercero <yctercero@users.noreply.github.com>
Summary
Fixes #147849
When changing number of rows in exception lists view from any page other than page 1.
Fix
bug_fix_147849.mov
Checklist