fix(plugin-chart-echarts): remove columns from formData (#901)

apache · Nov 26, 2021 · 1e8cb81 · 1e8cb81
1 parent c9e5d68
commit 1e8cb81
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/.../temporary_superset_ui/superset-ui/plugins/plugin-chart-echarts/src/BoxPlot/buildQuery.ts b/.../temporary_superset_ui/superset-ui/plugins/plugin-chart-echarts/src/BoxPlot/buildQuery.ts
@@ -45,6 +45,7 @@ export default function buildQuery(formData: BoxPlotQueryFormData) {
         ...baseQueryObject,
         is_timeseries: distributionColumns.length === 0,
         groupby: (groupby || []).concat(distributionColumns),
+        columns: [],
         post_processing: [
           {
             operation: 'boxplot',

diff --git a/...rary_superset_ui/superset-ui/plugins/plugin-chart-echarts/test/BoxPlot/buildQuery.test.ts b/...rary_superset_ui/superset-ui/plugins/plugin-chart-echarts/test/BoxPlot/buildQuery.test.ts
@@ -38,6 +38,7 @@ describe('BoxPlot buildQuery', () => {
     expect(query.is_timeseries).toEqual(true);
     expect(query.metrics).toEqual(['foo']);
     expect(query.groupby).toEqual(['bar']);
+    expect(query.columns).toEqual([]);
   });
 
   it('should build non-timeseries query object when columns is defined', () => {
@@ -46,5 +47,6 @@ describe('BoxPlot buildQuery', () => {
     expect(query.is_timeseries).toEqual(false);
     expect(query.metrics).toEqual(['foo']);
     expect(query.groupby).toEqual(['bar', 'qwerty']);
+    expect(query.columns).toEqual([]);
   });
 });