Skip to content

[APM] Fix ML links#22820

Merged
sorenlouv merged 1 commit intoelastic:masterfrom
sorenlouv:fix-ml-links
Sep 8, 2018
Merged

[APM] Fix ML links#22820
sorenlouv merged 1 commit intoelastic:masterfrom
sorenlouv:fix-ml-links

Conversation

@sorenlouv
Copy link
Copy Markdown
Contributor

A fix in #22592 which was supposed to fix links to ML jobs, didn't work.

@sorenlouv sorenlouv added the Team:APM - DEPRECATED Use Team:obs-ux-infra_services. label Sep 7, 2018
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/apm-ui

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jasonrhodes At least now it is based on the KibanaLink component. Is that ok with you?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Totally fine with me -- sorry I missed this not working in the other PR!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No worries. I should have manually tested it after making the change.

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@sorenlouv
Copy link
Copy Markdown
Contributor Author

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@sorenlouv sorenlouv merged commit 6efab01 into elastic:master Sep 8, 2018
@sorenlouv sorenlouv deleted the fix-ml-links branch September 8, 2018 12:49
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Sep 8, 2018
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Sep 8, 2018
sorenlouv added a commit that referenced this pull request Sep 8, 2018
…22851)

* [APM] Fix broken links (#22592)

* [APM] Fix broken links

* Add missing basepaths

* Remove basepath from getMlJobUrl

* [APM] Fix ML links (#22820)
sorenlouv added a commit that referenced this pull request Sep 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:APM - DEPRECATED Use Team:obs-ux-infra_services. v6.4.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants