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.

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] Edit job selection on data frame analytics results and map pages #130419

Merged

Conversation

alvarezmelissa87
Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 commented Apr 15, 2022

Summary

Fixes #129284

image

image

Checklist

Delete any items that are not applicable to this PR.

@alvarezmelissa87 alvarezmelissa87 self-assigned this Apr 15, 2022
@alvarezmelissa87 alvarezmelissa87 requested a review from a team as a code owner April 15, 2022 21:26
@alvarezmelissa87 alvarezmelissa87 added release_note:skip Skip the PR/issue when compiling release notes :ml Feature:Data Frame Analytics ML data frame analytics features labels Apr 15, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

<EuiButtonEmpty
size="xs"
iconType="pencil"
onClick={() => setIsIdSelectorFlyoutVisible(true)}
Copy link
Member

Choose a reason for hiding this comment

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

nit, could be

onClick={setIsIdSelectorFlyoutVisible.bind(null, true)}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated in 9b80e44

Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

@peteharverson peteharverson added release_note:enhancement and removed release_note:skip Skip the PR/issue when compiling release notes labels Apr 20, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
ml 1574 1575 +1

Async chunks

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

id before after diff
ml 3.3MB 3.3MB +1.1KB

History

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

cc @alvarezmelissa87

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested latest changes and LGTM

@alvarezmelissa87 alvarezmelissa87 merged commit 0d233e9 into elastic:main Apr 20, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Apr 20, 2022
@alvarezmelissa87 alvarezmelissa87 deleted the ml-dfa-edit-job-selection branch April 20, 2022 17:26
dmlemeshko pushed a commit to dmlemeshko/kibana that referenced this pull request May 5, 2022
…map pages (elastic#130419)

* add selection flyout control

* always allow selection flyout to be opened

* show flyout when no job is selected

* lint fix

* show badge when id selected
kertal pushed a commit to kertal/kibana that referenced this pull request May 24, 2022
…map pages (elastic#130419)

* add selection flyout control

* always allow selection flyout to be opened

* show flyout when no job is selected

* lint fix

* show badge when id selected
@lcawl lcawl changed the title [ML] Allow to edit job selection on data frame analytics results and map pages [ML] Edit job selection on data frame analytics results and map pages Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Data Frame Analytics ML data frame analytics features :ml release_note:enhancement v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ML] Allow to edit job selection on data frame analytics results and map pages
6 participants