You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2024/03/28 14:30:52 UTC
(superset) branch master updated: fix: Removes filter plugins from viz gallery (#27763)
This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new bbcb722e50 fix: Removes filter plugins from viz gallery (#27763)
bbcb722e50 is described below
commit bbcb722e50dd602d3f53fa88127c90110bea8763
Author: Michael S. Molina <70...@users.noreply.github.com>
AuthorDate: Thu Mar 28 11:30:45 2024 -0300
fix: Removes filter plugins from viz gallery (#27763)
---
superset-frontend/src/constants.ts | 8 ++++++++
.../explore/components/controls/VizTypeControl/index.tsx | 5 ++++-
superset-frontend/src/pages/ChartCreation/index.tsx | 6 ++++--
.../src/visualizations/presets/MainPreset.js | 15 ++++++++++-----
4 files changed, 26 insertions(+), 8 deletions(-)
diff --git a/superset-frontend/src/constants.ts b/superset-frontend/src/constants.ts
index b64cf91375..ed7d62dbe8 100644
--- a/superset-frontend/src/constants.ts
+++ b/superset-frontend/src/constants.ts
@@ -189,3 +189,11 @@ export const DEFAULT_COMMON_BOOTSTRAP_DATA: CommonBootstrapData = {
export const DEFAULT_BOOTSTRAP_DATA: BootstrapData = {
common: DEFAULT_COMMON_BOOTSTRAP_DATA,
};
+
+export enum FilterPlugins {
+ Select = 'filter_select',
+ Range = 'filter_range',
+ Time = 'filter_time',
+ TimeColumn = 'filter_timecolumn',
+ TimeGrain = 'filter_timegrain',
+}
diff --git a/superset-frontend/src/explore/components/controls/VizTypeControl/index.tsx b/superset-frontend/src/explore/components/controls/VizTypeControl/index.tsx
index bdfcf1e40a..fb2ecad2ff 100644
--- a/superset-frontend/src/explore/components/controls/VizTypeControl/index.tsx
+++ b/superset-frontend/src/explore/components/controls/VizTypeControl/index.tsx
@@ -28,6 +28,7 @@ import { usePluginContext } from 'src/components/DynamicPlugins';
import Modal from 'src/components/Modal';
import { noOp } from 'src/utils/common';
import getBootstrapData from 'src/utils/getBootstrapData';
+import { FilterPlugins } from 'src/constants';
import VizTypeGallery, {
MAX_ADVISABLE_VIZ_GALLERY_WIDTH,
} from './VizTypeGallery';
@@ -43,7 +44,9 @@ interface VizTypeControlProps {
}
const bootstrapData = getBootstrapData();
-const denyList: string[] = bootstrapData.common.conf.VIZ_TYPE_DENYLIST || [];
+const denyList: string[] = (
+ bootstrapData.common.conf.VIZ_TYPE_DENYLIST || []
+).concat(Object.values(FilterPlugins));
const metadataRegistry = getChartMetadataRegistry();
export const VIZ_TYPE_CONTROL_TEST_ID = 'viz-type-control';
diff --git a/superset-frontend/src/pages/ChartCreation/index.tsx b/superset-frontend/src/pages/ChartCreation/index.tsx
index f3a602dbc9..f621519102 100644
--- a/superset-frontend/src/pages/ChartCreation/index.tsx
+++ b/superset-frontend/src/pages/ChartCreation/index.tsx
@@ -27,7 +27,7 @@ import {
t,
} from '@superset-ui/core';
import { getUrlParam } from 'src/utils/urlUtils';
-import { URL_PARAMS } from 'src/constants';
+import { FilterPlugins, URL_PARAMS } from 'src/constants';
import { Link, withRouter, RouteComponentProps } from 'react-router-dom';
import Button from 'src/components/Button';
import { AsyncSelect, Steps } from 'src/components';
@@ -60,7 +60,9 @@ const ESTIMATED_NAV_HEIGHT = 56;
const ELEMENTS_EXCEPT_VIZ_GALLERY = ESTIMATED_NAV_HEIGHT + 250;
const bootstrapData = getBootstrapData();
-const denyList: string[] = bootstrapData.common.conf.VIZ_TYPE_DENYLIST || [];
+const denyList: string[] = (
+ bootstrapData.common.conf.VIZ_TYPE_DENYLIST || []
+).concat(Object.values(FilterPlugins));
const StyledContainer = styled.div`
${({ theme }) => `
diff --git a/superset-frontend/src/visualizations/presets/MainPreset.js b/superset-frontend/src/visualizations/presets/MainPreset.js
index 698663e1da..f77f8c037b 100644
--- a/superset-frontend/src/visualizations/presets/MainPreset.js
+++ b/superset-frontend/src/visualizations/presets/MainPreset.js
@@ -77,6 +77,7 @@ import {
} from 'src/filters/components';
import { PivotTableChartPlugin as PivotTableChartPluginV2 } from '@superset-ui/plugin-chart-pivot-table';
import { HandlebarsChartPlugin } from '@superset-ui/plugin-chart-handlebars';
+import { FilterPlugins } from 'src/constants';
import TimeTableChartPlugin from '../TimeTable';
export default class MainPreset extends Preset {
@@ -157,11 +158,15 @@ export default class MainPreset extends Preset {
new EchartsWaterfallChartPlugin().configure({
key: 'waterfall',
}),
- new SelectFilterPlugin().configure({ key: 'filter_select' }),
- new RangeFilterPlugin().configure({ key: 'filter_range' }),
- new TimeFilterPlugin().configure({ key: 'filter_time' }),
- new TimeColumnFilterPlugin().configure({ key: 'filter_timecolumn' }),
- new TimeGrainFilterPlugin().configure({ key: 'filter_timegrain' }),
+ new SelectFilterPlugin().configure({ key: FilterPlugins.Select }),
+ new RangeFilterPlugin().configure({ key: FilterPlugins.Range }),
+ new TimeFilterPlugin().configure({ key: FilterPlugins.Time }),
+ new TimeColumnFilterPlugin().configure({
+ key: FilterPlugins.TimeColumn,
+ }),
+ new TimeGrainFilterPlugin().configure({
+ key: FilterPlugins.TimeGrain,
+ }),
new EchartsTreeChartPlugin().configure({ key: 'tree_chart' }),
new EchartsSunburstChartPlugin().configure({ key: 'sunburst_v2' }),
new HandlebarsChartPlugin().configure({ key: 'handlebars' }),