-
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
[New Platform] Expose uiSettings on server side #44994
Comments
Pinging @elastic/kibana-platform |
Needed for #29215 |
Related: #22779 |
Needed for migrating the Graph server APIs |
There are a couple of existing plugins that would also be blocked by this (graph, siem, infra, dashboard_mode) The server UISettingsClient will have to be exposed through the request context and use the request scoped SavedObjectsClient so that advanced settings remain space aware. |
Correct. |
As the very first step, we can migrate legacy service to TypeScript. If SavedObjects and Logging with metadata aren't available soon, we can hack and provide legacy service via |
We haven't prioritized logging metadata very high in our backlog. Maybe we need to re-evaluate? Do you know what else needs that feature? |
Only Security & Spaces plugins waits for it, other calls are done by the platform & optimizer. |
minimum functionality is implemented. possible improvements are moved to #48925 |
Plan
Nice to do:
sinon
,expect
. update testing infrastructure for ui_settings #51266defaults
term as non-descriptive UI settings move to NP #47590 (comment)Original post:
uiSettings is available on core in client side, but I think it makes sense to expose a similar service server side. The data plugin could use it, we'd like users to be able to use the search API, and if they leave certain settings off, we'll default to the ones in uiSettings.
cc @lukasolson
The text was updated successfully, but these errors were encountered: