Skip to content

[8.x] [Security Solution] Implement refactoring remark from PR #201731 (#204022)#205905

Merged
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-204022
Jan 9, 2025
Merged

[8.x] [Security Solution] Implement refactoring remark from PR #201731 (#204022)#205905
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-204022

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

… (elastic#204022)

## Summary

In the PR elastic#201731 for ticket elastic#180660 @banderror advised to refactor code
in that PR to better separate the concerns (business logic from
components). This is the implementation of that review
[remark](https://github.com/elastic/kibana/pull/201731/files#r1860492191).

Recording:

https://github.com/user-attachments/assets/471a0986-bcdb-4611-ab1a-bdcbe5151f47

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Nikita Indik <nikita.indik@elastic.co>
(cherry picked from commit 20eb87d)
@kibanamachine kibanamachine merged commit fada2e8 into elastic:8.x Jan 9, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 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 19.2MB 19.2MB -6.0B

History

cc @jkelas

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