-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ML] Register ML and Transforms for the kibana home page #62499
[ML] Register ML and Transforms for the kibana home page #62499
Conversation
Pinging @elastic/ml-ui (:ml) |
62d7e53
to
773ba85
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM, didn't test locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and LGTM
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* [ML] Register ML for the kibana home page * adds transforms * tiny refactor * updating comments Co-authored-by: Elastic Machine <[email protected]>
* [ML] Register ML for the kibana home page * adds transforms * tiny refactor * updating comments Co-authored-by: Elastic Machine <[email protected]>
…2602) * [ML] Register ML for the kibana home page * adds transforms * tiny refactor * updating comments Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
…2603) * [ML] Register ML for the kibana home page * adds transforms * tiny refactor * updating comments Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Registering ML and Transforms so they appear on the kibana home page.
It's worth noting that currently Transforms does not have its own icon and so I have used the management app's generic cog icon.
Fixes #62478