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

[Stateful sidenav] Update stack management landing page #191735

Merged
merged 14 commits into from
Sep 24, 2024

Conversation

sebelga
Copy link
Contributor

@sebelga sebelga commented Aug 29, 2024

In this PR I've removed the management cards in the stack management landing page and used the classic landing page.

This changes applies to all 3 solutions, only for the new side navigation in stateful.

Fixes https://github.com/elastic/kibana-team/issues/1063

Screenshots

Screenshot 2024-09-20 at 15 25 54 Screenshot 2024-09-20 at 15 26 05

@sebelga sebelga self-assigned this Aug 29, 2024
@sebelga sebelga added Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) release_note:skip Skip the PR/issue when compiling release notes labels Aug 29, 2024
@sebelga sebelga marked this pull request as ready for review August 29, 2024 13:20
@sebelga sebelga requested a review from a team as a code owner August 29, 2024 13:20
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@sebelga sebelga requested a review from semd August 29, 2024 13:27
@sebelga sebelga marked this pull request as draft August 29, 2024 15:57
@sebelga sebelga force-pushed the stateful-sidenav/management-landing-page branch 2 times, most recently from e6f9830 to 92f993b Compare September 19, 2024 13:20
@sebelga
Copy link
Contributor Author

sebelga commented Sep 19, 2024

/ci

3 similar comments
@sebelga
Copy link
Contributor Author

sebelga commented Sep 19, 2024

/ci

@sebelga
Copy link
Contributor Author

sebelga commented Sep 19, 2024

/ci

@sebelga
Copy link
Contributor Author

sebelga commented Sep 19, 2024

/ci

@sebelga sebelga force-pushed the stateful-sidenav/management-landing-page branch from df4ea50 to c059a5b Compare September 20, 2024 09:02
@sebelga
Copy link
Contributor Author

sebelga commented Sep 20, 2024

/ci

@sebelga sebelga force-pushed the stateful-sidenav/management-landing-page branch from c059a5b to 1af2486 Compare September 20, 2024 14:25
@sebelga sebelga marked this pull request as ready for review September 20, 2024 14:26
@sebelga sebelga requested a review from a team as a code owner September 20, 2024 14:26
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-management Observability Management User Experience Team labels Sep 20, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

Copy link
Contributor

@semd semd left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@sabarasaba sabarasaba left a comment

Choose a reason for hiding this comment

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

kibana management changes lgtm, this looks awesome! 🚀

@kc13greiner
Copy link
Contributor

@sebelga Code LGTM, but I cant remember how to get the new Nav to show up 😅 Can you remind me?

@sebelga
Copy link
Contributor Author

sebelga commented Sep 23, 2024

I cant remember how to get the new Nav to show up 😅 Can you remind me?

@kc13greiner you need to add the following in the kibana.dev.yml

xpack.spaces.experimental.forceSolutionVisibility: true

And then create a space with the solution you want.

Copy link
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

LGTM!

@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 24, 2024

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: faa0cfb
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-191735-faa0cfbfe99f

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
management 128 130 +2
securitySolutionEss 118 117 -1
total +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/core-chrome-browser 101 102 +1
@kbn/shared-ux-chrome-navigation 26 29 +3
total +4

Async chunks

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

id before after diff
enterpriseSearch 2.6MB 2.6MB +22.0B
management 46.1KB 48.3KB +2.2KB
navigation 23.5KB 23.8KB +360.0B
observability 467.3KB 467.3KB +22.0B
security 591.2KB 591.3KB +43.0B
securitySolutionEss 58.7KB 62.1KB +3.4KB
serverless 24.2KB 24.5KB +360.0B
total +6.4KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 455.0KB 455.6KB +612.0B
management 10.7KB 10.7KB +44.0B
securitySolutionEss 16.5KB 11.6KB -4.9KB
total -4.3KB
Unknown metric groups

API count

id before after diff
@kbn/core-chrome-browser 207 208 +1
@kbn/shared-ux-chrome-navigation 35 38 +3
total +4

History

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

cc @sebelga

Comment on lines +72 to +76
<FormattedMessage
id="management.landing.solution.header"
defaultMessage="Stack Management {version}"
values={{ version: kibanaVersion }}
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we exclude kibana version particularly for serverless projects, to be in line with the direction we've taken for other instances where kibana version previously would get displayed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The empty prompt is not shown in serverless. In serverless we display the management cards.

Copy link
Contributor

@eokoneyo eokoneyo 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, works for all 3 solutions

@sebelga
Copy link
Contributor Author

sebelga commented Sep 24, 2024

Thanks for the review @eokoneyo !

@sebelga sebelga merged commit 92f1320 into elastic:main Sep 24, 2024
47 checks passed
@sebelga sebelga deleted the stateful-sidenav/management-landing-page branch September 24, 2024 13:01
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 24, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
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 Sep 24, 2024
… (#193867)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Stateful sidenav] Update stack management landing page
(#191735)](#191735)

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

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

<!--BACKPORT [{"author":{"name":"Sébastien
Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2024-09-24T13:01:02Z","message":"[Stateful
sidenav] Update stack management landing page
(#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management"],"title":"[Stateful
sidenav] Update stack management landing
page","number":191735,"url":"https://github.com/elastic/kibana/pull/191735","mergeCommit":{"message":"[Stateful
sidenav] Update stack management landing page
(#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/191735","number":191735,"mergeCommit":{"message":"[Stateful
sidenav] Update stack management landing page
(#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824"}}]}]
BACKPORT-->

Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
neptunian pushed a commit to neptunian/kibana that referenced this pull request Oct 1, 2024
…c#191735) (elastic#193867)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Stateful sidenav] Update stack management landing page
(elastic#191735)](elastic#191735)

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

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

<!--BACKPORT [{"author":{"name":"Sébastien
Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2024-09-24T13:01:02Z","message":"[Stateful
sidenav] Update stack management landing page
(elastic#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management"],"title":"[Stateful
sidenav] Update stack management landing
page","number":191735,"url":"https://github.com/elastic/kibana/pull/191735","mergeCommit":{"message":"[Stateful
sidenav] Update stack management landing page
(elastic#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/191735","number":191735,"mergeCommit":{"message":"[Stateful
sidenav] Update stack management landing page
(elastic#191735)","sha":"92f13200194e2faf35aa95a21e95d39323b2e824"}}]}]
BACKPORT-->

Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
semd added a commit that referenced this pull request Oct 2, 2024
## Summary

Since the Stack Management landing page [has been
dropped](#191735) in the new
"Solution navigation" in stateful (ESS) environments, we need to include
some of the missing Security Solution links in the left navigation
flyout panel.

The PR also contains the removal of the "Recent" section from the left
navigation, this is a SecuritySolution-specific product/design decision.

#### Adding the missing links to the Stack Management left navigation
panel flyout

- Entity Risk Score (Alerts and Insights)
- Asset Criticality (Alerts and Insights)
- Graph (Kibana)
- Canvas (Kibana)
- Map (Kibana)
- Visualiztion Library (Kibana)


![nav1](https://github.com/user-attachments/assets/8864237f-22a0-4efd-9cce-cccf1a6ab710)
![nav2](https://github.com/user-attachments/assets/0b223782-6ebc-453e-8f0d-37fa6854a780)


#### Removing the "Recent" links item from the navigation.

Before
<img width="277" alt="old"
src="https://github.com/user-attachments/assets/0eaccd3c-04e9-4838-bae9-e44e4fa0cec9">

After
<img width="277" alt="Captura de pantalla 2024-10-02 a les 13 32 55"
src="https://github.com/user-attachments/assets/53266f93-de2b-4d31-9935-5a793b6c99e8">

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 2, 2024
…4384)

## Summary

Since the Stack Management landing page [has been
dropped](elastic#191735) in the new
"Solution navigation" in stateful (ESS) environments, we need to include
some of the missing Security Solution links in the left navigation
flyout panel.

The PR also contains the removal of the "Recent" section from the left
navigation, this is a SecuritySolution-specific product/design decision.

#### Adding the missing links to the Stack Management left navigation
panel flyout

- Entity Risk Score (Alerts and Insights)
- Asset Criticality (Alerts and Insights)
- Graph (Kibana)
- Canvas (Kibana)
- Map (Kibana)
- Visualiztion Library (Kibana)

![nav1](https://github.com/user-attachments/assets/8864237f-22a0-4efd-9cce-cccf1a6ab710)
![nav2](https://github.com/user-attachments/assets/0b223782-6ebc-453e-8f0d-37fa6854a780)

#### Removing the "Recent" links item from the navigation.

Before
<img width="277" alt="old"
src="https://github.com/user-attachments/assets/0eaccd3c-04e9-4838-bae9-e44e4fa0cec9">

After
<img width="277" alt="Captura de pantalla 2024-10-02 a les 13 32 55"
src="https://github.com/user-attachments/assets/53266f93-de2b-4d31-9935-5a793b6c99e8">

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 4da439d)
kibanamachine added a commit that referenced this pull request Oct 2, 2024
) (#194720)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[SecuritySolution] Stateful new solution nav improvements
(#194384)](#194384)

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

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

<!--BACKPORT [{"author":{"name":"Sergi
Massaneda","email":"sergi.massaneda@elastic.co"},"sourceCommit":{"committedDate":"2024-10-02T14:45:38Z","message":"[SecuritySolution]
Stateful new solution nav improvements (#194384)\n\n##
Summary\r\n\r\nSince the Stack Management landing page [has
been\r\ndropped](#191735) in the
new\r\n\"Solution navigation\" in stateful (ESS) environments, we need
to include\r\nsome of the missing Security Solution links in the left
navigation\r\nflyout panel.\r\n\r\nThe PR also contains the removal of
the \"Recent\" section from the left\r\nnavigation, this is a
SecuritySolution-specific product/design decision.\r\n\r\n#### Adding
the missing links to the Stack Management left navigation\r\npanel
flyout\r\n\r\n- Entity Risk Score (Alerts and Insights)\r\n- Asset
Criticality (Alerts and Insights)\r\n- Graph (Kibana)\r\n- Canvas
(Kibana)\r\n- Map (Kibana)\r\n- Visualiztion Library
(Kibana)\r\n\r\n\r\n![nav1](https://github.com/user-attachments/assets/8864237f-22a0-4efd-9cce-cccf1a6ab710)\r\n![nav2](https://github.com/user-attachments/assets/0b223782-6ebc-453e-8f0d-37fa6854a780)\r\n\r\n\r\n####
Removing the \"Recent\" links item from the
navigation.\r\n\r\nBefore\r\n<img width=\"277\"
alt=\"old\"\r\nsrc=\"https://github.com/user-attachments/assets/0eaccd3c-04e9-4838-bae9-e44e4fa0cec9\">\r\n\r\nAfter\r\n<img
width=\"277\" alt=\"Captura de pantalla 2024-10-02 a les 13 32
55\"\r\nsrc=\"https://github.com/user-attachments/assets/53266f93-de2b-4d31-9935-5a793b6c99e8\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"4da439df876ffa148b33f82c1acc6b725b37829e","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:
SecuritySolution","backport:prev-minor","v8.16.0"],"title":"[SecuritySolution]
Stateful new solution nav
improvements","number":194384,"url":"https://github.com/elastic/kibana/pull/194384","mergeCommit":{"message":"[SecuritySolution]
Stateful new solution nav improvements (#194384)\n\n##
Summary\r\n\r\nSince the Stack Management landing page [has
been\r\ndropped](#191735) in the
new\r\n\"Solution navigation\" in stateful (ESS) environments, we need
to include\r\nsome of the missing Security Solution links in the left
navigation\r\nflyout panel.\r\n\r\nThe PR also contains the removal of
the \"Recent\" section from the left\r\nnavigation, this is a
SecuritySolution-specific product/design decision.\r\n\r\n#### Adding
the missing links to the Stack Management left navigation\r\npanel
flyout\r\n\r\n- Entity Risk Score (Alerts and Insights)\r\n- Asset
Criticality (Alerts and Insights)\r\n- Graph (Kibana)\r\n- Canvas
(Kibana)\r\n- Map (Kibana)\r\n- Visualiztion Library
(Kibana)\r\n\r\n\r\n![nav1](https://github.com/user-attachments/assets/8864237f-22a0-4efd-9cce-cccf1a6ab710)\r\n![nav2](https://github.com/user-attachments/assets/0b223782-6ebc-453e-8f0d-37fa6854a780)\r\n\r\n\r\n####
Removing the \"Recent\" links item from the
navigation.\r\n\r\nBefore\r\n<img width=\"277\"
alt=\"old\"\r\nsrc=\"https://github.com/user-attachments/assets/0eaccd3c-04e9-4838-bae9-e44e4fa0cec9\">\r\n\r\nAfter\r\n<img
width=\"277\" alt=\"Captura de pantalla 2024-10-02 a les 13 32
55\"\r\nsrc=\"https://github.com/user-attachments/assets/53266f93-de2b-4d31-9935-5a793b6c99e8\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"4da439df876ffa148b33f82c1acc6b725b37829e"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194384","number":194384,"mergeCommit":{"message":"[SecuritySolution]
Stateful new solution nav improvements (#194384)\n\n##
Summary\r\n\r\nSince the Stack Management landing page [has
been\r\ndropped](#191735) in the
new\r\n\"Solution navigation\" in stateful (ESS) environments, we need
to include\r\nsome of the missing Security Solution links in the left
navigation\r\nflyout panel.\r\n\r\nThe PR also contains the removal of
the \"Recent\" section from the left\r\nnavigation, this is a
SecuritySolution-specific product/design decision.\r\n\r\n#### Adding
the missing links to the Stack Management left navigation\r\npanel
flyout\r\n\r\n- Entity Risk Score (Alerts and Insights)\r\n- Asset
Criticality (Alerts and Insights)\r\n- Graph (Kibana)\r\n- Canvas
(Kibana)\r\n- Map (Kibana)\r\n- Visualiztion Library
(Kibana)\r\n\r\n\r\n![nav1](https://github.com/user-attachments/assets/8864237f-22a0-4efd-9cce-cccf1a6ab710)\r\n![nav2](https://github.com/user-attachments/assets/0b223782-6ebc-453e-8f0d-37fa6854a780)\r\n\r\n\r\n####
Removing the \"Recent\" links item from the
navigation.\r\n\r\nBefore\r\n<img width=\"277\"
alt=\"old\"\r\nsrc=\"https://github.com/user-attachments/assets/0eaccd3c-04e9-4838-bae9-e44e4fa0cec9\">\r\n\r\nAfter\r\n<img
width=\"277\" alt=\"Captura de pantalla 2024-10-02 a les 13 32
55\"\r\nsrc=\"https://github.com/user-attachments/assets/53266f93-de2b-4d31-9935-5a793b6c99e8\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"4da439df876ffa148b33f82c1acc6b725b37829e"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Sergi Massaneda <sergi.massaneda@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-management Observability Management User Experience Team Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants