Skip to content

[8.6] [Discover] Fix Discover navigation from Lens embeddable (#147000)#147608

Merged
dimaanj merged 3 commits intoelastic:8.6from
dimaanj:backport/8.6/pr-147000
Dec 15, 2022
Merged

[8.6] [Discover] Fix Discover navigation from Lens embeddable (#147000)#147608
dimaanj merged 3 commits intoelastic:8.6from
dimaanj:backport/8.6/pr-147000

Conversation

@dimaanj
Copy link
Contributor

@dimaanj dimaanj commented Dec 15, 2022

Backport

This will backport the following commits from main to 8.6:

Questions ?

Please refer to the Backport tool documentation

## Summary

Fixes elastic#146761

This PR fixes navigation to Discover from Lens embeddable.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

Co-authored-by: Marco Liberati <dej611@users.noreply.github.com>
(cherry picked from commit 1f04bf8)

# Conflicts:
#	x-pack/plugins/lens/public/trigger_actions/open_in_discover_action.test.ts
@dimaanj dimaanj added the backport This PR is a backport of another PR label Dec 15, 2022
@dimaanj dimaanj enabled auto-merge (squash) December 15, 2022 11:36
@kibana-ci
Copy link

💚 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
lens 1.3MB 1.3MB +94.0B

Page load bundle

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

id before after diff
unifiedSearch 50.5KB 50.6KB +46.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

History

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

@dimaanj dimaanj merged commit 66d0f55 into elastic:8.6 Dec 15, 2022
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