-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[SecuritySolution] Restore lens table columns when Reset group by field clicked. #156393
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
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.
Desk tested and LGTM! Just a small comment about a typo in LensDataTableAttributes
.
x-pack/plugins/security_solution/public/common/components/visualization_actions/types.ts
Outdated
Show resolved
Hide resolved
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @angorayc |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ld clicked. (elastic#156393) ## Summary issue: elastic#155859 Steps to reproduce: Please find the issue above. `Reset group by field` action should: 1. Revert `Group by` and `Group by top` to default fields `kibana.alert.rule.name` and `host.name` 2. Unhide all the columns in the table. ### Checklist Delete any items that are not applicable to this PR. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios (cherry picked from commit db02784)
…by field clicked. (#156393) (#156454) # Backport This will backport the following commits from `main` to `8.8`: - [[SecuritySolution] Restore lens table columns when Reset group by field clicked. (#156393)](#156393) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Angela Chuang","email":"6295984+angorayc@users.noreply.github.com"},"sourceCommit":{"committedDate":"2023-05-02T22:02:12Z","message":"[SecuritySolution] Restore lens table columns when Reset group by field clicked. (#156393)\n\n## Summary\r\n\r\nissue: #155859 to reproduce: Please find the issue above.\r\n\r\n`Reset group by field` action should:\r\n1. Revert `Group by` and `Group by top` to default fields\r\n`kibana.alert.rule.name` and `host.name`\r\n2. Unhide all the columns in the table.\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"db02784f98c6c41cbb0a0ccbc4895d67b698c4aa","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:Threat Hunting","Team: SecuritySolution","Team:Threat Hunting:Explore","v8.8.0","Feature:Lens Charts","v8.9.0"],"number":156393,"url":"#156393 Restore lens table columns when Reset group by field clicked. (#156393)\n\n## Summary\r\n\r\nissue: #155859 to reproduce: Please find the issue above.\r\n\r\n`Reset group by field` action should:\r\n1. Revert `Group by` and `Group by top` to default fields\r\n`kibana.alert.rule.name` and `host.name`\r\n2. Unhide all the columns in the table.\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"db02784f98c6c41cbb0a0ccbc4895d67b698c4aa"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"#156393 Restore lens table columns when Reset group by field clicked. (#156393)\n\n## Summary\r\n\r\nissue: #155859 to reproduce: Please find the issue above.\r\n\r\n`Reset group by field` action should:\r\n1. Revert `Group by` and `Group by top` to default fields\r\n`kibana.alert.rule.name` and `host.name`\r\n2. Unhide all the columns in the table.\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"db02784f98c6c41cbb0a0ccbc4895d67b698c4aa"}}]}] BACKPORT--> Co-authored-by: Angela Chuang <6295984+angorayc@users.noreply.github.com>
Summary
issue: #155859
Steps to reproduce: Please find the issue above.
Reset group by field
action should:Group by
andGroup by top
to default fieldskibana.alert.rule.name
andhost.name
Checklist
Delete any items that are not applicable to this PR.