Skip to content
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] Expandable flyout - add data view title and query bar to rule preview panel #164650

Merged

Conversation

christineweng
Copy link
Contributor

@christineweng christineweng commented Aug 23, 2023

Summary

This PR adds dataViewTitle to rule preview panel -> define section when data view is available in a rule. This addresses #164529.

How to test

  • Create a rule with data view
  • Generate some alerts, go to alerts page, pick a alert and open expandable flyout
  • Go to About, open Rule summary
  • Expand Define section, the content should match the define section in rule details page for that rule

image

Checklist

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 15.7MB 15.7MB +1.5KB
Unknown metric groups

References to deprecated APIs

id before after diff
securitySolution 641 646 +5

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @christineweng

Copy link
Contributor

@PhilippeOberti PhilippeOberti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the quick fix!

@christineweng christineweng merged commit 1bb4a52 into elastic:main Aug 24, 2023
9 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 24, 2023
… bar to rule preview panel (elastic#164650)

## Summary

This PR adds `dataViewTitle` to rule preview panel -> define section
when data view is available in a rule. This addresses
elastic#164529.

**How to test**
- Create a rule with data view
- Generate some alerts, go to alerts page, pick a alert and open
expandable flyout
- Go to About, open `Rule summary`
- Expand `Define` section, the content should match the define section
in rule details page for that rule

![image](https://github.com/elastic/kibana/assets/18648970/30aeff6b-547a-4b68-be87-9b52ab58501b)

### Checklist

- [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 1bb4a52)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.10

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Aug 24, 2023
…d query bar to rule preview panel (#164650) (#164717)

# Backport

This will backport the following commits from `main` to `8.10`:
- [[Security Solution] Expandable flyout - add data view title and query
bar to rule preview panel
(#164650)](#164650)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"christineweng","email":"18648970+christineweng@users.noreply.github.com"},"sourceCommit":{"committedDate":"2023-08-24T13:44:09Z","message":"[Security
Solution] Expandable flyout - add data view title and query bar to rule
preview panel (#164650)\n\n## Summary\r\n\r\nThis PR adds
`dataViewTitle` to rule preview panel -> define section\r\nwhen data
view is available in a rule. This
addresses\r\nhttps://github.com//issues/164529.\r\n\r\n**How
to test**\r\n- Create a rule with data view\r\n- Generate some alerts,
go to alerts page, pick a alert and open\r\nexpandable flyout\r\n- Go to
About, open `Rule summary`\r\n- Expand `Define` section, the content
should match the define section\r\nin rule details page for that
rule\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/18648970/30aeff6b-547a-4b68-be87-9b52ab58501b)\r\n\r\n\r\n###
Checklist\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":"1bb4a5269242ac2a9ac7cadf4d535e0ea7b5e720","branchLabelMapping":{"^v8.11.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Threat
Hunting:Investigations","v8.10.0","v8.11.0"],"number":164650,"url":"https://github.com/elastic/kibana/pull/164650","mergeCommit":{"message":"[Security
Solution] Expandable flyout - add data view title and query bar to rule
preview panel (#164650)\n\n## Summary\r\n\r\nThis PR adds
`dataViewTitle` to rule preview panel -> define section\r\nwhen data
view is available in a rule. This
addresses\r\nhttps://github.com//issues/164529.\r\n\r\n**How
to test**\r\n- Create a rule with data view\r\n- Generate some alerts,
go to alerts page, pick a alert and open\r\nexpandable flyout\r\n- Go to
About, open `Rule summary`\r\n- Expand `Define` section, the content
should match the define section\r\nin rule details page for that
rule\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/18648970/30aeff6b-547a-4b68-be87-9b52ab58501b)\r\n\r\n\r\n###
Checklist\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":"1bb4a5269242ac2a9ac7cadf4d535e0ea7b5e720"}},"sourceBranch":"main","suggestedTargetBranches":["8.10"],"targetPullRequestStates":[{"branch":"8.10","label":"v8.10.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.11.0","labelRegex":"^v8.11.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/164650","number":164650,"mergeCommit":{"message":"[Security
Solution] Expandable flyout - add data view title and query bar to rule
preview panel (#164650)\n\n## Summary\r\n\r\nThis PR adds
`dataViewTitle` to rule preview panel -> define section\r\nwhen data
view is available in a rule. This
addresses\r\nhttps://github.com//issues/164529.\r\n\r\n**How
to test**\r\n- Create a rule with data view\r\n- Generate some alerts,
go to alerts page, pick a alert and open\r\nexpandable flyout\r\n- Go to
About, open `Rule summary`\r\n- Expand `Define` section, the content
should match the define section\r\nin rule details page for that
rule\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/18648970/30aeff6b-547a-4b68-be87-9b52ab58501b)\r\n\r\n\r\n###
Checklist\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":"1bb4a5269242ac2a9ac7cadf4d535e0ea7b5e720"}}]}]
BACKPORT-->

Co-authored-by: christineweng <18648970+christineweng@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants