-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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] Fix links to dashboards in Lens created anomaly detection jobs #160156
[ML] Fix links to dashboards in Lens created anomaly detection jobs #160156
Conversation
Pinging @elastic/ml-ui (:ml) |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
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 other than the typo, LGTM!
// embeddable may have not been in a dashboard | ||
// and my not have been given a title as it is unsaved. |
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.
Typo - may
instead of my
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 ⚡
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…lastic#160156) When creating a job from a Lens visualisation, a link back to the original dashboard is added to the job's custom URLs. This has recently become broken due to the dashboard ID retrieved from the lens embeddable not matching the real dashboard ID. Rather than rely on this ID, the actual dashboard is retrieved from kibana using the dashboard service and the ID acquired from there. The dashboard service also contains the dashboard url locator which we can use rather than use the `share` plugin. (cherry picked from commit cf92a67)
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
4 similar comments
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
1 similar comment
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
When creating a job from a Lens visualisation, a link back to the original dashboard is added to the job's custom URLs.
This has recently become broken due to the dashboard ID retrieved from the lens embeddable not matching the real dashboard ID.
Rather than rely on this ID, the actual dashboard is retrieved from kibana using the dashboard service and the ID acquired from there.
The dashboard service also contains the dashboard url locator which we can use rather than use the
share
plugin.