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
[Discover] Add kibana services provider for embeddable #121621
[Discover] Add kibana services provider for embeddable #121621
Conversation
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
@elasticmachine merge upstream |
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.
This PR fixes the bug 👍 tested and confirmed in Safari, MacOS. Could you add a functional test for it? we need to back port this. many thx. BTW about functional testing of the expand functionality, I'm currently working on making an existing test unflaky #107311
This test wouldn't cover the expand functionality on a dashboard, but it can be used as a template to not having to reinvent the wheel :)
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
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.
LGTM, didn't test again, but there was just a functional test added, thx a lot for it 👍
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @dmitriynj |
💔 Some backports could not be created
How to fixRe-run the backport manually:
Questions ?Please refer to the Backport tool documentation |
* [Discover] add kibana services provider for embeddable * [Discover] add functional test * [Discover] fix functional test * [Discover] fix data grid test * [Discover] go to discover main page before each test * [Discover] remove redundant navigation to discover Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit f5aa068) # Conflicts: # src/plugins/discover/public/application/embeddable/saved_search_grid.tsx
* [Discover] add kibana services provider for embeddable * [Discover] add functional test * [Discover] fix functional test * [Discover] fix data grid test * [Discover] go to discover main page before each test * [Discover] remove redundant navigation to discover Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit f5aa068) # Conflicts: # src/plugins/discover/public/application/embeddable/saved_search_grid.tsx
* [Discover] add kibana services provider for embeddable * [Discover] add functional test * [Discover] fix functional test * [Discover] fix data grid test * [Discover] go to discover main page before each test * [Discover] remove redundant navigation to discover Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
#122702) * [Discover] Add kibana services provider for embeddable (#121621) * [Discover] add kibana services provider for embeddable * [Discover] add functional test * [Discover] fix functional test * [Discover] fix data grid test * [Discover] go to discover main page before each test * [Discover] remove redundant navigation to discover Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit f5aa068) # Conflicts: # src/plugins/discover/public/application/embeddable/saved_search_grid.tsx * [Discover] fix imports * [Discover] return data grid expand action * [Discover] fix functional test Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
… (#122701) * [Discover] Add kibana services provider for embeddable (#121621) * [Discover] add kibana services provider for embeddable * [Discover] add functional test * [Discover] fix functional test * [Discover] fix data grid test * [Discover] go to discover main page before each test * [Discover] remove redundant navigation to discover Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit f5aa068) # Conflicts: # src/plugins/discover/public/application/embeddable/saved_search_grid.tsx * [Discover] fix imports * [Discover] return data grid expand action * [Discover] fix functional test Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
Fixes #121183
This PR adds kibana services provider for embeddable.