-
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
[Graph] Raw Documents drilldown broken #18027
Comments
Original comment by @markharwood: Another issue I see in master is that the Settings panel no longer detects Kibana urls when they are pasted into the drill-down-url edit box. It is not easy for users to turn a Kibana URL into a template due to rison encoding etc so as a convenience we auto-detect and offer to replace the relevant part of Kibana urls that have the empty "*" query by detecting this string:
unfortunately the default query string (at least in bar chart visualizations) now looks like this:
The code that would need changing is this line from app.js:
Note that this breakage is likely to happen again and is symptomatic of a bigger issue - there is no visualization-independent means of passing in a query definition via URL so each client app like Graph has to poke around inside the internal rison-encoded state of each viz to wire these connections together. However, there's an even bigger issue here in that we really shouldn't be messing around asking users to build a system by wiring viz URLs together in this way in the first place. We should have infrastructure to allow selections in one Kibana visualization to be re-used as queries or subject matter (think |
Original comment by @spalger:
When you choose the
Raw Documents
drill down in graph it redirects to discover and[object Object]
is selected as the index, which causes a warning to be displayed.The text was updated successfully, but these errors were encountered: