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

[SecuritySolution][Navigation] Prevent initial re-render using project nav #201431

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

semd
Copy link
Contributor

@semd semd commented Nov 22, 2024

Summary

Prevents an initial re-render when the project navigation style is set (serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security Solution.

@semd semd added release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. backport:version Backport to applied version labels v8.17.0 v8.18.0 v8.16.2 labels Nov 22, 2024
@semd semd self-assigned this Nov 22, 2024
@semd semd requested review from a team as code owners November 22, 2024 16:44
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@semd semd enabled auto-merge (squash) November 22, 2024 19:27
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 25, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Detection Engine - Security Solution Cypress Tests #5 / EQL query rules Editing rule with non-blocking query validation errors should allow user to save a rule and show confirmation modal when data source does not exist should allow user to save a rule and show confirmation modal when data source does not exist
  • [job] [logs] Jest Tests #14 / RootPrivilegesCallout should render callout requiring root privileges
  • [job] [logs] FTR Configs #73 / Visualizations - Group 2 lens app - Agg based Vis Open in Lens Heatmap should convert to Lens

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 13.3MB 13.3MB +40.0B

History

cc @semd

@semd semd disabled auto-merge November 25, 2024 17:47
@semd semd enabled auto-merge (squash) November 25, 2024 17:48
Copy link
Contributor

@michaelolo24 michaelolo24 left a comment

Choose a reason for hiding this comment

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

Tested locally and didn't see any odd rendering behavior with the initial undefined setting and no more re-renders. Thank you for making this change! 🚀

@semd semd merged commit d3ee297 into elastic:main Nov 25, 2024
46 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.17, 8.x

https://github.com/elastic/kibana/actions/runs/12015929027

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 25, 2024
…t nav (elastic#201431)

## Summary

Prevents an initial re-render when the `project` navigation style is set
(serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security
Solution.

---------

Co-authored-by: Michael Olorunnisola <[email protected]>
(cherry picked from commit d3ee297)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 25, 2024
…t nav (elastic#201431)

## Summary

Prevents an initial re-render when the `project` navigation style is set
(serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security
Solution.

---------

Co-authored-by: Michael Olorunnisola <[email protected]>
(cherry picked from commit d3ee297)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 25, 2024
…t nav (elastic#201431)

## Summary

Prevents an initial re-render when the `project` navigation style is set
(serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security
Solution.

---------

Co-authored-by: Michael Olorunnisola <[email protected]>
(cherry picked from commit d3ee297)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.17
8.x

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 Nov 25, 2024
… project nav (#201431) (#201666)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[SecuritySolution][Navigation] Prevent initial re-render using
project nav (#201431)](#201431)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Massaneda","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-25T17:56:28Z","message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:
SecuritySolution","backport:version","v8.17.0","v8.18.0","v8.16.2"],"title":"[SecuritySolution][Navigation]
Prevent initial re-render using project
nav","number":201431,"url":"https://github.com/elastic/kibana/pull/201431","mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.x","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201431","number":201431,"mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Sergi Massaneda <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 25, 2024
… project nav (#201431) (#201667)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[SecuritySolution][Navigation] Prevent initial re-render using
project nav (#201431)](#201431)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Massaneda","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-25T17:56:28Z","message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:
SecuritySolution","backport:version","v8.17.0","v8.18.0","v8.16.2"],"title":"[SecuritySolution][Navigation]
Prevent initial re-render using project
nav","number":201431,"url":"https://github.com/elastic/kibana/pull/201431","mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.x","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201431","number":201431,"mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Sergi Massaneda <[email protected]>
kibanamachine added a commit that referenced this pull request Nov 25, 2024
…project nav (#201431) (#201668)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[SecuritySolution][Navigation] Prevent initial re-render using
project nav (#201431)](#201431)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Massaneda","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-25T17:56:28Z","message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:
SecuritySolution","backport:version","v8.17.0","v8.18.0","v8.16.2"],"title":"[SecuritySolution][Navigation]
Prevent initial re-render using project
nav","number":201431,"url":"https://github.com/elastic/kibana/pull/201431","mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.x","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201431","number":201431,"mergeCommit":{"message":"[SecuritySolution][Navigation]
Prevent initial re-render using project nav (#201431)\n\n##
Summary\r\n\r\nPrevents an initial re-render when the `project`
navigation style is set\r\n(serverless and new solution nav for
stateful).\r\n\r\nThis re-render was affecting all the top-level pages
in Security\r\nSolution.\r\n\r\n---------\r\n\r\nCo-authored-by: Michael
Olorunnisola
<[email protected]>","sha":"d3ee297d9fc5e007d7333816074b02463dd0c646"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Sergi Massaneda <[email protected]>
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
…t nav (elastic#201431)

## Summary

Prevents an initial re-render when the `project` navigation style is set
(serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security
Solution.

---------

Co-authored-by: Michael Olorunnisola <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…t nav (elastic#201431)

## Summary

Prevents an initial re-render when the `project` navigation style is set
(serverless and new solution nav for stateful).

This re-render was affecting all the top-level pages in Security
Solution.

---------

Co-authored-by: Michael Olorunnisola <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.16.2 v8.17.0 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants