-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Inventory] Fix Open Explore in Discover link in a new tab #207346
Conversation
/ci |
/ci |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes LGTM 💯 Thanks for the fix, just a nit ⬇️
x-pack/solutions/observability/plugins/inventory/public/hooks/use_discover_redirect.ts
Outdated
Show resolved
Hide resolved
/ci |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
cc @rmyz |
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
Starting backport for target branches: 8.x |
…07346) ## Summary Closes elastic#207064 This PR fixes the Explore in Discover link to be able to open it in a new tab using `dataViewSpec: dataView.toMinimalSpec()`. ## How to test it 1. Enable `entityCentricExperience` feature flag 2. Run some synthtrace scenario, for example `node scripts/synthtrace infra_docker_containers` 3. Click into an entity group and select Explore in Discover in any entity. 4. You should be redirected to the data view correctly by clicking or opening in a new tab --------- Co-authored-by: kibanamachine <[email protected]> (cherry picked from commit 131a543)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…7346) (#207434) # Backport This will backport the following commits from `main` to `8.x`: - [[Inventory] Fix Open Explore in Discover link in a new tab (#207346)](#207346) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Sergi Romeu","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-21T19:54:55Z","message":"[Inventory] Fix Open Explore in Discover link in a new tab (#207346)\n\n## Summary\n\nCloses #207064\n\nThis PR fixes the Explore in Discover link to be able to open it in a\nnew tab using `dataViewSpec: dataView.toMinimalSpec()`.\n\n## How to test it\n1. Enable `entityCentricExperience` feature flag\n2. Run some synthtrace scenario, for example `node scripts/synthtrace\ninfra_docker_containers`\n3. Click into an entity group and select Explore in Discover in any\nentity.\n4. You should be redirected to the data view correctly by clicking or\nopening in a new tab\n\n---------\n\nCo-authored-by: kibanamachine <[email protected]>","sha":"131a543eb280e45d3d4493da96e29fbd2e5df87c","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","backport:prev-minor","Team:obs-ux-infra_services"],"title":"[Inventory] Fix Open Explore in Discover link in a new tab","number":207346,"url":"https://github.com/elastic/kibana/pull/207346","mergeCommit":{"message":"[Inventory] Fix Open Explore in Discover link in a new tab (#207346)\n\n## Summary\n\nCloses #207064\n\nThis PR fixes the Explore in Discover link to be able to open it in a\nnew tab using `dataViewSpec: dataView.toMinimalSpec()`.\n\n## How to test it\n1. Enable `entityCentricExperience` feature flag\n2. Run some synthtrace scenario, for example `node scripts/synthtrace\ninfra_docker_containers`\n3. Click into an entity group and select Explore in Discover in any\nentity.\n4. You should be redirected to the data view correctly by clicking or\nopening in a new tab\n\n---------\n\nCo-authored-by: kibanamachine <[email protected]>","sha":"131a543eb280e45d3d4493da96e29fbd2e5df87c"}},"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/207346","number":207346,"mergeCommit":{"message":"[Inventory] Fix Open Explore in Discover link in a new tab (#207346)\n\n## Summary\n\nCloses #207064\n\nThis PR fixes the Explore in Discover link to be able to open it in a\nnew tab using `dataViewSpec: dataView.toMinimalSpec()`.\n\n## How to test it\n1. Enable `entityCentricExperience` feature flag\n2. Run some synthtrace scenario, for example `node scripts/synthtrace\ninfra_docker_containers`\n3. Click into an entity group and select Explore in Discover in any\nentity.\n4. You should be redirected to the data view correctly by clicking or\nopening in a new tab\n\n---------\n\nCo-authored-by: kibanamachine <[email protected]>","sha":"131a543eb280e45d3d4493da96e29fbd2e5df87c"}}]}] BACKPORT--> Co-authored-by: Sergi Romeu <[email protected]>
Summary
Closes #207064
This PR fixes the Explore in Discover link to be able to open it in a new tab using
dataViewSpec: dataView.toMinimalSpec()
.How to test it
entityCentricExperience
feature flagnode scripts/synthtrace infra_docker_containers