-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Search][Homepage] Indices Card #188491
[Search][Homepage] Indices Card #188491
Conversation
Added a config value to set if getting index stats is available. for ES3 we will not be able to fetch index stats so we need an easy way to disable this in routes that fetch index data.
Added the get indices route with support for stack and serverless.
f900d36
to
f6d1dbb
Compare
/ci |
Removed the import of the ent-search context. Instead we will need to share search context in another way that doesn't result in circular references :/
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.
Core changes 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.
LGTM
💚 Build Succeeded
Metrics [docs]Module Count
Public APIs missing comments
Async chunks
Public APIs missing exports
Page load bundle
History
To update your PR or re-run it, just comment with: |
Summary
Introduces the Indices card for the (feature flagged) search homepage.
Testing
To enable the Search Home page execute the following command in the Dev Console:
You can then disable the feature flag with the following command:
Screenshots
Empty State
Stack w/ Index
Checklist
Delete any items that are not applicable to this PR.