-
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
[Enterprise Search] Add EuiButtonEmptyTo component #85213
[Enterprise Search] Add EuiButtonEmptyTo component #85213
Conversation
This PR adds another wrapper component for internal routing, the EuiButtonEmptyTo component.
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.
Nice, thanks Scotty!!! I think EuiCardTo might be our only/last helper after this, woo!
💚 Build SucceededMetrics [docs]Async chunks
Distributable file count
To update your PR or re-run it, just comment with: |
…or-disable-searchable-snapshot-fields * 'master' of github.com:elastic/kibana: (29 commits) HTTP CRUD+ API for Index Patterns (elastic#83576) Don't list packages in devDependencies already listed in dependencies (elastic#85120) Remove unused devDependencies (elastic#85121) [ILM] Reposition form toggles (elastic#85143) [APM] Make sure jest script can be run from anywhere (elastic#85111) Add EuiButtonEmptyTo components (elastic#85213) skip flaky suite (elastic#85216) skip flaky suite (elastic#83775) skip flaky suite (elastic#83774) skip flaky suite (elastic#83773) skip flaky suite (elastic#83793) skip flaky suite (elastic#85215) skip flaky suite (elastic#85217) [Usage Collection] Kibana Overview Page UI Metrics (elastic#81937) [Alerting UI] Replaced AlertsContextProvider with KibanaContextProvider and exposed components in API (elastic#84604) skip flaky suite (elastic#85208) [Security Solutions][Detection Engine] Fixes cypress errors by using latest signals mapping (elastic#84600) Small fixes to Kibana search bar (elastic#85079) Change link text to say Fleet (elastic#85083) [Metrics UI] Refactor Process List API to fetch only top processes (elastic#84716) ... # Conflicts: # x-pack/plugins/index_lifecycle_management/public/application/sections/edit_policy/components/phases/hot_phase/hot_phase.tsx # x-pack/plugins/index_lifecycle_management/public/application/sections/edit_policy/components/phases/shared_fields/searchable_snapshot_field/searchable_snapshot_field.tsx
@scottybollinger Do you mind backporting this to 7.x as well so App Search can use this? 🙇♀️ |
Just did. Got this yesterday or it would have been done already: ✔ Cherry-picking: Add EuiButtonEmptyTo components (#85213)
✔ Pushing branch "scottybollinger:backport/7.x/pr-85213"
✖ Creating pull request
Could not create pull request: API rate limit exceeded for user ID 1869731. |
This PR adds another wrapper component for internal routing, the EuiButtonEmptyTo component.
Whoa!! Interesting, never seen that before... Scotty, you DDoSing GitHub again?? |
Summary
This PR adds another wrapper component for internal routing, the
EuiButtonEmptyTo
component.Checklist