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

Use docLinks API for APM doc links #61880

Merged
merged 3 commits into from
Mar 30, 2020
Merged

Conversation

smith
Copy link
Contributor

@smith smith commented Mar 30, 2020

Remove packageInfo (from ui/metadata package from the legacy platform) and replace with core.dockLinks in the ElasticDocsLink component.

Fixes cases where documentation links were not working because the version we were using in the URL is not the same version as the what the docs use.

Fixes #56453
Fixes #60945

Remove `packageInfo` (from ui/metadata package from the legacy platform) and replace with `core.dockLinks` in the `ElasticDocsLink` component.

Fixes elastic#56453
Fixes elastic#60945
@smith smith added bug Fixes for quality problems that affect the customer experience release_note:fix v7.7.0 labels Mar 30, 2020
@smith smith requested a review from a team as a code owner March 30, 2020 19:01
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

Copy link
Contributor

@ogupte ogupte left a comment

Choose a reason for hiding this comment

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

Much better now, thank you!

@smith smith merged commit 5808f86 into elastic:master Mar 30, 2020
@smith smith deleted the nls/doc-links-fix branch March 30, 2020 23:34
smith added a commit to smith/kibana that referenced this pull request Mar 30, 2020
* Use docLinks API for APM doc links

Remove `packageInfo` (from ui/metadata package from the legacy platform) and replace with `core.dockLinks` in the `ElasticDocsLink` component.

Fixes elastic#56453
Fixes elastic#60945
smith added a commit to smith/kibana that referenced this pull request Mar 30, 2020
* Use docLinks API for APM doc links

Remove `packageInfo` (from ui/metadata package from the legacy platform) and replace with `core.dockLinks` in the `ElasticDocsLink` component.

Fixes elastic#56453
Fixes elastic#60945
smith added a commit that referenced this pull request Mar 31, 2020
* Use docLinks API for APM doc links

Remove `packageInfo` (from ui/metadata package from the legacy platform) and replace with `core.dockLinks` in the `ElasticDocsLink` component.

Fixes #56453
Fixes #60945
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 1, 2020
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

smith added a commit that referenced this pull request Apr 2, 2020
* Use docLinks API for APM doc links

Remove `packageInfo` (from ui/metadata package from the legacy platform) and replace with `core.dockLinks` in the `ElasticDocsLink` component.

Fixes #56453
Fixes #60945

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 2, 2020
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 release_note:fix v7.7.0
Projects
None yet
3 participants