Repository navigation
feat: remove box plot - #582
dmitriy-kostianetskiy wants to merge 8 commits into
Conversation
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
- Added `addDefaultChart` method to `MosaicDashboardContext` for default chart creation. - Wrapped `RendererComponent` in `DashboardPanelErrorBoundary` in `MosaicDashboardPanel` for error handling. - Updated `MosaicDashboardProfilerPanelRenderer` to use `ProfilerPanelRendererProps`. - Refactored `createMosaicDashboardVgPlotPanelConfig` to accept `VgPlotChartConfig` directly. - Modified `MosaicDashboardToolbar` to trigger `addDefaultChart` instead of `openBuilder`. - Improved `ChartSettings` component structure with a new context for better state management. - Removed deprecated chart settings components (e.g., `BoxPlotSettings`, `BubbleSettings`, etc.) and replaced them with a dynamic settings approach. - Introduced `DynamicChartSettings` for handling chart-specific settings dynamically based on selected chart type. - Updated `generateMosaicChartSpec` to include error handling and improved logging. - Cleaned up imports and exports across various files for better organization and clarity. Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
might be superseded by #588 |
|
not relevant anymore, fixed here: #588 |
Box Plot chart not supported.