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
[Lens] collapse by for partition charts #140336
[Lens] collapse by for partition charts #140336
Conversation
@@ -701,6 +701,7 @@ export type VisualizationDimensionGroupConfig = SharedDimensionProps & { | |||
groupId: string; | |||
accessors: AccessorConfig[]; | |||
supportsMoreColumns: boolean; | |||
dimensionsTooMany?: number; | |||
/** If required, a warning will appear if accessors are empty */ | |||
required?: boolean; | |||
requiredMinDimensionCount?: number; |
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.
Follow-up: it looks like requiredMinDimensionCount
is no longer used anywhere after #139214
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
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 everything works fine - great work! I left one comment about and i18n phrase, but I think no second review is necessary. LGTM once that's fixed
x-pack/plugins/lens/public/editor_frame_service/editor_frame/config_panel/layer_panel.tsx
Outdated
Show resolved
Hide resolved
…andrewctate/kibana into 137057/collapse-by-for-partition-charts
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Async chunks
History
To update your PR or re-run it, just comment with: |
Summary
Resolve #137057
Screen.Recording.2022-09-08.at.4.59.26.PM.mov
Checklist
Delete any items that are not applicable to this PR.