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

[reporting] implement k7Breadcrumbs #26724

Merged
merged 2 commits into from
Dec 6, 2018

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Dec 5, 2018

Summary

This PR updates the reporting routes to provide k7Breadcrumbs used by the new header navigation. See #25884 for general information about the integration with the router and #25689 for the breadcrumb taxonomy

image

Checklist

@spalger spalger added review v7.0.0 (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Dec 5, 2018
@spalger spalger requested a review from cchaos December 5, 2018 18:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@elasticmachine

This comment has been minimized.

@tsullivan
Copy link
Member

I don't know if I'm missing something, or missed a step somehow, but I don't see new breadcrumbs when I pull this branch:
image

@cchaos
Copy link
Contributor

cchaos commented Dec 5, 2018

@tsullivan You'll have to turn on the K7 header from advanced settings.

Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

LGTM

@elastic elastic deleted a comment from elasticmachine Dec 5, 2018
@elasticmachine

This comment has been minimized.

@elasticmachine

This comment has been minimized.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit 425c8d5 into elastic:master Dec 6, 2018
@spalger spalger mentioned this pull request Dec 6, 2018
20 tasks
@spalger spalger deleted the implement/k7-breadcrumbs/reporting branch August 18, 2020 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead review Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants