-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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 URL generator #67937
Discover URL generator #67937
Conversation
Pinging @elastic/kibana-app-arch (Team:AppArch) |
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 LGTM, but didn't test. (no way to test from UI)
Does it make sense to use it in our current example drilldown to discover? Then we could actually click this through UI.
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 LGTM, tested the example in Chrome 83, MacOs 10.14.6, could add a discover drilldown in the example, link is added, drilldown available, link works 👍
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* feat: 🎸 stub discover_enhanced plugin * feat: 🎸 add Discover URL generator * chore: 🤖 remove x-pack plugin * fix: 🐛 fix types in URL generator * test: 💍 setup test file for Discover URL generator * feat: 🎸 expose Discover URL generator in start life-cycle * feat: 🎸 add ability to specify saved search ID in URL generator * docs: ✏️ add JSDoc for Discover URL generator * fix: 🐛 set correctly global filters in Discover URL generator * docs: ✏️ remove wrong comment in JsDoc * style: 💄 format single arg arrow function with parens * chore: 🤖 disable toggles in Dicover sample drilldown * feat: 🎸 use Discover URL generator in example plugin * test: 💍 add urlGenerator mock * test: 💍 add .registerUrlGenerator() test mock * test: 💍 correct Karma mock for "share" plugin URL generator # Conflicts: # x-pack/examples/ui_actions_enhanced_examples/kibana.json # x-pack/examples/ui_actions_enhanced_examples/public/plugin.ts
Summary
Adds URL generator for Discover app. This is needed for #62652 and #60227
Checklist
Delete any items that are not applicable to this PR.
For maintainers