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

[Logs / Metrics UI] [NP followup] Fix navigation within the same app via the Kibana nav bar #58112

Closed
Kerry350 opened this issue Feb 20, 2020 · 2 comments · Fixed by #61667
Closed
Assignees
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@Kerry350
Copy link
Contributor

Kerry350 commented Feb 20, 2020

Currently if you are already within the metrics or logs app, and then try to click the same app again within the Kibana side navigation, the route will not re-render (so if you were on the /metrics/explorer page and clicked the Metrics icon in the Kibana navigation the /metrics route won't render).

This is due to #53692 and will be fixed with #56705. Once #56705 is completed, the implementation of this ticket should be to switch to the new ScopedHistory.

@Kerry350 Kerry350 added Feature:Metrics UI Metrics UI feature Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Feb 20, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@joshdover
Copy link
Contributor

ScopedHistory has been merged, should be able to switch over now in Metrics and Logs routers 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants