Skip to content

Use docLinks API for APM doc links - #61880

Merged
smith merged 3 commits into
elastic:masterfrom
smith:nls/doc-links-fix
Mar 30, 2020
Merged

smith merged 3 commits into
elastic:masterfrom
smith:nls/doc-links-fix

Conversation

@smith

@smith smith commented Mar 30, 2020

Copy link
Copy Markdown
Contributor

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 March 30, 2020 19:01
@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@ogupte ogupte left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* 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
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

Development

Successfully merging this pull request may close these issues.

[APM][UI] Broken link in Transaction Metadata page [APM] Documentation links using ElasticDocsLink.tsx don't always work

3 participants