-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Maps] fix application state filters transfer from other kibana application to maps application #73516
Conversation
…cation to maps application
Pinging @elastic/kibana-gis (Team:Geo) |
@@ -102,14 +102,6 @@ export class MapsAppView extends React.Component { | |||
this._globalSyncChangeMonitorSubscription.unsubscribe(); | |||
} | |||
|
|||
// Clean up app state filters |
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.
No longer need to clean-up on unmount since app state filters gets cleaned up on mount
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededBuild metricsasync chunks size
History
To update your PR or re-run it, just comment with: |
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.
lgtm! thanks for the clean up!
- code review
…cation to maps application (elastic#73516) * [Maps] fix application state filters transfer from other kibana application to maps application * clean up comment Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
…cation to maps application (elastic#73516) * [Maps] fix application state filters transfer from other kibana application to maps application * clean up comment Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
…cation to maps application (#73516) (#73829) * [Maps] fix application state filters transfer from other kibana application to maps application * clean up comment Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
…cation to maps application (#73516) (#73830) * [Maps] fix application state filters transfer from other kibana application to maps application * clean up comment Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Fixes #73354
The regression was caused as a side effect of moving to new platform. Prior to new platform, switching to maps would trigger a refresh and clean filterManager instance. Without the page reload, filterManager contains application state filters from other Kibana applications. This PR just clears application state filters from filter manager when setting up application state syncing.