-
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
discover chart and url do not update when default index is changed #2480
Comments
Not sure if this is a bug, or rather just a behavior. We store the last state of each app. The index pattern is included in the state, so when you click back to discover, you're really just reloading the state you were in. If you were to somehow end up in a default discover state you'd get the default index pattern. |
if I delete the url after changing default index and after selecting discover tab, the prev default index renders in the chart but the url shows the new default |
it seems the last state in the url and the rendered chart can be different. |
It looks like I stumbled upon a similar issue. Go to settings, select another index pattern, click on Discovery, and contents from the previous index is shown, it shows also the previous index name in the field list. I couldn't find an obvious way to force it to change, other than deleting the query string from the url. |
Confirmed, if you manually reset the state of the app by modifying the query-string, then it resets to the default index. However, is that the work-flow that we expect our users to follow? IMO, As a user, if I go into the settings and change the selected default index, I expect that the next time I navigate to the Discover tab, the new default index will be selected. I expect the application to do whatever it needs to do to establish that state. |
We considered adapting this in a few ways:
That's when we decided we didn't like any of our options, and that this probably isn't going to happen. Additional rationale:
If you think you've got a solution that works when taking this all into account feel free to open another issue |
The text was updated successfully, but these errors were encountered: