Skip to content
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

[Search Sessions][Dashboard] Clear search session when navigating from dashboard route #89749

Merged
merged 2 commits into from
Feb 1, 2021

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Jan 29, 2021

Summary

Currently dashboard calls session.clear() only when dashboard app is unmounted:
https://github.com/elastic/kibana/blob/master/src/plugins/dashboard/public/application/dashboard_router.tsx#L222

The session isn't clear when navigating within the dashboard app which leads to a stale session indicator:
Screenshot 2021-01-29 at 14 07 02

This pr adds session.clear() when dashboard app route unmounts.
I also had to leave the original clear because of #82877

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@Dosant Dosant added bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features Feature:Search Querying infrastructure in Kibana v7.12.0 v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Jan 29, 2021
@Dosant Dosant marked this pull request as ready for review January 29, 2021 17:25
@Dosant Dosant requested a review from a team as a code owner January 29, 2021 17:25
Copy link
Contributor

@ThomThomson ThomThomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code only review, but change LGTM. Good to add a functional test for it as well!

@Dosant
Copy link
Contributor Author

Dosant commented Feb 1, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 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
dashboard 160.7KB 160.9KB +112.0B

History

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features Feature:Search Querying infrastructure in Kibana release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants