This is an automated email from the ASF dual-hosted git repository. diegopucci pushed a commit to branch fix/dashboard-filter-first-value in repository https://gitbox.apache.org/repos/asf/superset.git
commit e6feadf8289d54e2523ea40fce797778d5ee74c2 Author: geido <[email protected]> AuthorDate: Tue Mar 28 16:54:55 2023 +0200 Remove redundant useEffect --- .../src/filters/components/Select/SelectFilterPlugin.tsx | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx index d3cb2b4dea..a04606212c 100644 --- a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx +++ b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx @@ -129,7 +129,6 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) { enableEmptyFilter && !inverseSelection && !values?.length; const suffix = inverseSelection && values?.length ? t(' (excluded)') : ''; - dispatchDataMask({ type: 'filterState', __cache: filterState, @@ -290,13 +289,9 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) { updateDataMask, data, groupby, - JSON.stringify(filterState), + JSON.stringify(filterState.value), ]); - useEffect(() => { - updateDataMask(filterState.value); - }, [JSON.stringify(filterState.value)]); - useEffect(() => { setDataMask(dataMask); }, [JSON.stringify(dataMask)]);
