-
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
Reorganize new/existing pipeline management screens into tabs #172027
Conversation
💚 Build Succeeded
Metrics [docs]Async chunks
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.
LGTM!
Please add team label too. Also you don't need backport:skip
all you need is version labels. CI will figure backports from them.
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
...erprise_search_content/components/search_index/pipelines/ml_inference/configure_pipeline.tsx
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.
LGTM
Summary
Split Configure step’s components in pipeline creation into two tabs:
Keep the split components' content as-is, they are out of scope for this task.
Remove the left hand side panels (“Create or select a pipeline”, “Select a trained ML Model” and blurbs underneath), and make the tab full width.
Also remove the ELSER text expansion callout from the flyout.
Checklist
For maintainers