-
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
[NP] Move visTypeXy plugin #63449
[NP] Move visTypeXy plugin #63449
Conversation
Pinging @elastic/kibana-app (Team:KibanaApp) |
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.
Should be disabled by default.
expressions: npStart.plugins.expressions, | ||
visualizations: npStart.plugins.visualizations, | ||
export const config = { | ||
schema: schema.object({ enabled: schema.boolean({ defaultValue: true }) }), |
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.
default value should false
.
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.
Ahh, sure, thanks!
Updated!
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
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.
Tested and LGTM - code is called in the right places 👍
* Move vis_type_xy into NP * Substitute usage in vis_type_vislib * Disable plugin by default # Conflicts: # .github/CODEOWNERS
* Move vis_type_xy into NP * Substitute usage in vis_type_vislib * Disable plugin by default
Summary
Summarize your PR. If it involves visual changes include a screenshot or gif.
Checklist
Delete any items that are not applicable to this PR.
For maintainers