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.

Sign up for GitHub

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

Jump to bottom

[7.7] Use docLinks API for APM doc links (#61880) #61909

Merged
merged 2 commits into from
Apr 2, 2020
Merged

[7.7] Use docLinks API for APM doc links (#61880) #61909

merged 2 commits into from
Apr 2, 2020

Conversation

smith
Copy link
Contributor

@smith smith commented Mar 30, 2020

Backports the following commits to 7.7:

@smith
Use docLinks API for APM doc links (elastic#61880)
5de1a5f 
* 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 smith added the backport label Mar 30, 2020
@smith
Copy link
Contributor Author

smith commented Mar 31, 2020

retest

2 similar comments
@smith
Copy link
Contributor Author

smith commented Mar 31, 2020

retest

@smith
Copy link
Contributor Author

smith commented Apr 1, 2020

retest

@spalger spalger assigned spalger and unassigned spalger Apr 1, 2020
@spalger spalger added backport and removed backport labels Apr 1, 2020
@smith
Copy link
Contributor Author

smith commented Apr 2, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@smith smith merged commit 02565fc into elastic:7.7 Apr 2, 2020
@smith smith deleted the backport/7.7/pr-61880 branch April 2, 2020 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@smith @kibanamachine @spalger @elasticmachine