Skip to content

[Move @kbn/config-schema to server] dashboard#189584

Merged
afharo merged 1 commit intoelastic:mainfrom
afharo:move-config-schema-to-server/dashboard
Jul 31, 2024
Merged

[Move @kbn/config-schema to server] dashboard#189584
afharo merged 1 commit intoelastic:mainfrom
afharo:move-config-schema-to-server/dashboard

Conversation

@afharo
Copy link
Member

@afharo afharo commented Jul 30, 2024

Summary

Part of #189476.

We want to avoid @kbn/config-schema from leaking to the browser, and config.ts should be part of the server logic.

For maintainers

@afharo afharo added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jul 30, 2024
@afharo afharo self-assigned this Jul 30, 2024
@afharo afharo requested a review from a team as a code owner July 30, 2024 22:37
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @afharo

@afharo afharo enabled auto-merge (squash) July 31, 2024 06:50
@afharo afharo merged commit 0305768 into elastic:main Jul 31, 2024
@afharo afharo deleted the move-config-schema-to-server/dashboard branch July 31, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants