-
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
[Move @kbn/config-schema
to server] @kbn/search-connectors
#191838
base: main
Are you sure you want to change the base?
[Move @kbn/config-schema
to server] @kbn/search-connectors
#191838
Conversation
Pinging @elastic/kibana-core (Team:Core) |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Public APIs missing exports
To update your PR or re-run it, just comment with: |
Pull request was converted to draft
🤖 Jobs for this PR can be triggered through checkboxes. 🚧
ℹ️ To trigger the CI, please tick the checkbox below 👇
|
Summary
Part of #189476.
We want to avoid
@kbn/config-schema
from leaking to the browser, and this package is using it while beingshared-common
.The class
ByteSizeValue
(used by a react component in this package) has been moved to its own package, and can now be imported from the UI without having to import from the server-side only package@kbn/config-schema
.For maintainers