-
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] Converts Recognizer job page to React #47429
Conversation
Pinging @elastic/ml-ui (:ml) |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
@elasticmachine merge upstream |
💔 Build Failed |
💚 Build Succeeded |
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.
Overall looking good! Just added some comments/questions.
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/create_result_callout.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/create_result_callout.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/create_result_callout.tsx
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/job_settings_form.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/job_settings_form.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/components/job_settings_form.tsx
Show resolved
Hide resolved
x-pack/legacy/plugins/ml/public/jobs/new_job_new/recognize/resolvers.ts
Outdated
Show resolved
Hide resolved
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 latest edits and functionally LGTM
x-pack/legacy/plugins/ml/public/jobs/new_job_new/common/components/time_range_picker.tsx
Outdated
Show resolved
Hide resolved
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.
Latest changes LGTM
💚 Build Succeeded |
* [ML] wip recognize job * [ML] error handling * [ML] layout enhancements * [ML] errors handling * [ML] fix badges alignment and error message * [ML] components split * [ML] load module from the component, reset support * [ML] validators * [ML] check view route and resolver * [ML] validation * [ML] remove angular code * [ML] change layout, fix patterns * [ML] job response fix * [ML] directive test * [ML] directive test * [ML] remove lookbehind regexp * [ML] fix IE11 layout issues * [ML] remove form tag, refactor to formState * [ML] aria labels, remove unused i18n * [ML] align kibana objects and reset button * [ML] column layout for jobs response * [ML] align icons and buttons * [ML] check existing kibana objects * [ML] don't display kibana objects panel if empty * [ML] result and reset button order * [ML] add EuiHorizontalRule * [ML] use time range and full index dat * [ML] no check icon for existing objects * [ML] toast for exception during a jobs setup * [ML] disable panels growth, advanced settings with described form * [ML] fix timeRange for results url * [ML] fix i18n * [ML] use EuiSwitch * [ML] PR remarks * [ML] job settings form component * [ML] i18n * [ML] onChange fix * [ML] custom hook for partial state update * [ML] jobGroups update * [ML] PR remarks * [ML] fix imports
💚 Build Succeeded |
* [ML] wip recognize job * [ML] error handling * [ML] layout enhancements * [ML] errors handling * [ML] fix badges alignment and error message * [ML] components split * [ML] load module from the component, reset support * [ML] validators * [ML] check view route and resolver * [ML] validation * [ML] remove angular code * [ML] change layout, fix patterns * [ML] job response fix * [ML] directive test * [ML] directive test * [ML] remove lookbehind regexp * [ML] fix IE11 layout issues * [ML] remove form tag, refactor to formState * [ML] aria labels, remove unused i18n * [ML] align kibana objects and reset button * [ML] column layout for jobs response * [ML] align icons and buttons * [ML] check existing kibana objects * [ML] don't display kibana objects panel if empty * [ML] result and reset button order * [ML] add EuiHorizontalRule * [ML] use time range and full index dat * [ML] no check icon for existing objects * [ML] toast for exception during a jobs setup * [ML] disable panels growth, advanced settings with described form * [ML] fix timeRange for results url * [ML] fix i18n * [ML] use EuiSwitch * [ML] PR remarks * [ML] job settings form component * [ML] i18n * [ML] onChange fix * [ML] custom hook for partial state update * [ML] jobGroups update * [ML] PR remarks * [ML] fix imports
Summary
Part of #18374. Recognizer main create job page to React.
Checklist
Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n supportDocumentation was added for features that require explanation or tutorialsFor maintainers
This was checked for breaking API changes and was labeled appropriatelyThis includes a feature addition or change that requires a release note and was labeled appropriately