Skip to content

[Oblt Onboarding] Add breadcrumbs for quick start flows#216470

Merged
mykolaharmash merged 2 commits intoelastic:mainfrom
mykolaharmash:191646-observability-onboarding-add-breadcrumbs-for-quickstart-flows
Apr 1, 2025
Merged

[Oblt Onboarding] Add breadcrumbs for quick start flows#216470
mykolaharmash merged 2 commits intoelastic:mainfrom
mykolaharmash:191646-observability-onboarding-add-breadcrumbs-for-quickstart-flows

Conversation

@mykolaharmash
Copy link
Contributor

@mykolaharmash mykolaharmash commented Mar 31, 2025 •
edited by kibanamachine
Loading

Closes #191646

This change adds breadcrumbs to the project navigation bar for every quickstart flow.

Classic Navigation Solution Navigation
CleanShot 2025-03-31 at 15 32 08@2x CleanShot 2025-03-31 at 15 31 13@2x

@mykolaharmash mykolaharmash requested a review from a team as a code owner March 31, 2025 13:35
@mykolaharmash mykolaharmash force-pushed the 191646-observability-onboarding-add-breadcrumbs-for-quickstart-flows branch from faf5001 to 10c6219 Compare March 31, 2025 13:38
@mykolaharmash mykolaharmash added the ci:project-deploy-observability Create an Observability project label Apr 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 1, 2025 •
edited
Loading

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: b097f9d
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-216470-b097f9dc882a

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #8 / should filter the table correctly

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observabilityOnboarding 228 229 +1

Async chunks

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

id before after diff
observabilityOnboarding 259.1KB 260.0KB +982.0B

History

flash1293
flash1293 approved these changes Apr 1, 2025
View reviewed changes
Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Tested and works well, LGTM

@mykolaharmash mykolaharmash merged commit fcc7244 into elastic:main Apr 1, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 1, 2025

Starting backport for target branches: 9.0

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

@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 1, 2025

💔 All backports failed

Status Branch Result
❌ 9.0 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.0:
- [Obs Onboarding] Add page rendering performance monitoring (#213769)

Manual backport

To create the backport manually run:

node scripts/backport --pr 216470

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 2, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 216470 locally

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 2, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 3, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 216470 locally

2 similar comments
@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 4, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 216470 locally

@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 7, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 216470 locally

@mykolaharmash mykolaharmash mentioned this pull request Apr 8, 2025
Closed
@mykolaharmash
Copy link
Contributor Author

mykolaharmash commented Apr 8, 2025

💚 All backports created successfully

Status Branch Result
✅ 9.0

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

Questions ?

Please refer to the Backport tool documentation

mykolaharmash added a commit to mykolaharmash/kibana that referenced this pull request Apr 8, 2025
@mykolaharmash
[Oblt Onboarding] Add breadcrumbs for quick start flows (elastic#216470)
ea2bd28 
Closes elastic#191646

This change adds breadcrumbs to the project navigation bar for every
quickstart flow.

| Classic Navigation | Solution Navigation |
| --- | --- |
| ![CleanShot 2025-03-31 at 15 32
08@2x](https://github.com/user-attachments/assets/d9489e9c-1df1-43f2-99ec-b82128a1be85)
| ![CleanShot 2025-03-31 at 15 31
13@2x](https://github.com/user-attachments/assets/28180e0b-da00-4773-b9e5-a4d7e1c97424)
|

(cherry picked from commit fcc7244)
@mykolaharmash mykolaharmash mentioned this pull request Apr 8, 2025
Merged
@mykolaharmash
Copy link
Contributor Author

mykolaharmash commented Apr 8, 2025

💚 All backports created successfully

Status Branch Result
✅ 8.18

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

Questions ?

Please refer to the Backport tool documentation

mykolaharmash added a commit to mykolaharmash/kibana that referenced this pull request Apr 8, 2025
@mykolaharmash
[Oblt Onboarding] Add breadcrumbs for quick start flows (elastic#216470)
d0521e1 
Closes elastic#191646

This change adds breadcrumbs to the project navigation bar for every
quickstart flow.

| Classic Navigation | Solution Navigation |
| --- | --- |
| ![CleanShot 2025-03-31 at 15 32
08@2x](https://github.com/user-attachments/assets/d9489e9c-1df1-43f2-99ec-b82128a1be85)
| ![CleanShot 2025-03-31 at 15 31
13@2x](https://github.com/user-attachments/assets/28180e0b-da00-4773-b9e5-a4d7e1c97424)
|

(cherry picked from commit fcc7244)
@kibanamachine
Copy link
Contributor

kibanamachine commented Apr 9, 2025

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

mykolaharmash added a commit that referenced this pull request Apr 9, 2025
@mykolaharmash
[8.18] [Oblt Onboarding] Add breadcrumbs for quick start flows (#216470…
5c5d8e3 
…) (#217495)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[Oblt Onboarding] Add breadcrumbs for quick start flows
(#216470)](#216470)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Mykola
Harmash","email":"mykola.harmash@gmail.com"},"sourceCommit":{"committedDate":"2025-04-01T11:49:09Z","message":"[Oblt
Onboarding] Add breadcrumbs for quick start flows (#216470)\n\nCloses
https://github.com/elastic/kibana/issues/191646\n\nThis change adds
breadcrumbs to the project navigation bar for every\nquickstart
flow.\n\n| Classic Navigation | Solution Navigation |\n| --- | --- |\n|
![CleanShot 2025-03-31 at 15
32\n08@2x](https://github.com/user-attachments/assets/d9489e9c-1df1-43f2-99ec-b82128a1be85)\n|
![CleanShot 2025-03-31 at 15
31\n13@2x](https://github.com/user-attachments/assets/28180e0b-da00-4773-b9e5-a4d7e1c97424)\n|","sha":"fcc7244da2b801ce5e59017a00e3ce19a20b01ce","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport
missing","backport:prev-minor","ci:project-deploy-observability","Feature:
Observability Onboarding"],"title":"[Oblt Onboarding] Add breadcrumbs
for quick start
flows","number":216470,"url":"https://github.com/elastic/kibana/pull/216470","mergeCommit":{"message":"[Oblt
Onboarding] Add breadcrumbs for quick start flows (#216470)\n\nCloses
https://github.com/elastic/kibana/issues/191646\n\nThis change adds
breadcrumbs to the project navigation bar for every\nquickstart
flow.\n\n| Classic Navigation | Solution Navigation |\n| --- | --- |\n|
![CleanShot 2025-03-31 at 15
32\n08@2x](https://github.com/user-attachments/assets/d9489e9c-1df1-43f2-99ec-b82128a1be85)\n|
![CleanShot 2025-03-31 at 15
31\n13@2x](https://github.com/user-attachments/assets/28180e0b-da00-4773-b9e5-a4d7e1c97424)\n|","sha":"fcc7244da2b801ce5e59017a00e3ce19a20b01ce"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[]}]
BACKPORT-->
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 9, 2025
mykolaharmash added a commit to mykolaharmash/kibana that referenced this pull request Apr 9, 2025
@mykolaharmash
Revert "[8.18] [Oblt Onboarding] Add breadcrumbs for quick start flows (
2ce728f 
elastic#216470) (elastic#217495)"

This reverts commit 5c5d8e3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@flash1293 flash1293 flash1293 approved these changes

Assignees

No one assigned

Labels

ci:project-deploy-observability Create an Observability project Feature: Observability Onboarding release_note:skip Skip the PR/issue when compiling release notes v8.18.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Observability Onboarding] Add breadcrumbs for quickstart flows

4 participants

@mykolaharmash @elasticmachine @kibanamachine @flash1293