Skip to content

[8.13] Fix threat intel edit filters (#179607)#180379

Merged
lgestc merged 1 commit intoelastic:8.13from
lgestc:backport/8.13/pr-179607
Apr 9, 2024
Merged

[8.13] Fix threat intel edit filters (#179607)#180379
lgestc merged 1 commit intoelastic:8.13from
lgestc:backport/8.13/pr-179607

Conversation

@lgestc
Copy link
Copy Markdown
Contributor

@lgestc lgestc commented Apr 9, 2024

Backport

This will backport the following commits from main to 8.13:

Questions ?

Please refer to the Backport tool documentation

## Summary

The following PR needs to be merged first:
elastic#178701

This fixes
elastic#174764 (comment)
and elastic#179030

**To reproduce:**

Add whatever filter in the Threat Intelligence (via table filter in),
then click it (in the top bar) - filter edit popover is not filled in
with data.

On Alerts page though, it is filled in correctly - and it should look
like that on TI:

![image](https://github.com/elastic/kibana/assets/11671118/0de5f076-83dd-48f3-810b-75d1572536e3)

(cherry picked from commit da69703)
@lgestc lgestc added the backport This PR is a backport of another PR label Apr 9, 2024
@lgestc lgestc enabled auto-merge (squash) April 9, 2024 14:14
@lgestc lgestc requested a review from PhilippeOberti April 9, 2024 14:16
Copy link
Copy Markdown
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.

backport LGTM for the Threat Hunting Investigations team!

@lgestc lgestc merged commit a0b2b29 into elastic:8.13 Apr 9, 2024
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #10 / useGetCurrentUserProfile shows a toast error message when an error occurs in the response

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
threatIntelligence 56.9KB 57.1KB +148.0B

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants