Skip to content

[ML][7.10] Fix zoom missing in Anomaly detection URLs - #86400

Merged
qn895 merged 4 commits into
elastic:7.10from
qn895:fix-7.10-zoom-params
Dec 23, 2020
Merged

qn895 merged 4 commits into
elastic:7.10from
qn895:fix-7.10-zoom-params

Conversation

@qn895

@qn895 qn895 commented Dec 17, 2020

Copy link
Copy Markdown
Member

Summary

This PR is the 7.10 equivalent of #86182. It also fixes an issue with the recently accessed items adding extra parts to the url, causing the link to redirect incorrectly.

Checklist

@qn895 qn895 self-assigned this Dec 17, 2020
@qn895 qn895 added the bug Fixes for quality problems that affect the customer experience label Dec 18, 2020
@qn895 qn895 added v8.0.0 and removed bug Fixes for quality problems that affect the customer experience labels Dec 18, 2020
@qn895
qn895 marked this pull request as ready for review December 18, 2020 01:13
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@peteharverson peteharverson 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.

LGTM

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 11.3MB 11.3MB +234.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
ml 65.7KB 65.7KB +25.0B

History

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

@darnautov darnautov 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.

LGTM

@qn895
qn895 merged commit cca59ba into elastic:7.10 Dec 23, 2020
@qn895
qn895 deleted the fix-7.10-zoom-params branch December 23, 2020 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants