Skip to content

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
@Dosant Dosant requested a review from ThomThomson 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