From aee94b39baaceb51cb6042188fd7f4e753266396 Mon Sep 17 00:00:00 2001 From: "Hugh A. Miles II" Date: Wed, 15 Nov 2023 18:05:18 -0500 Subject: [PATCH] fix(tag): update state to clear form on success (#25934) --- .../src/features/tags/TagModal.tsx | 21 ++++++++++++------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/superset-frontend/src/features/tags/TagModal.tsx b/superset-frontend/src/features/tags/TagModal.tsx index 4339d69130..a0ac8636a5 100644 --- a/superset-frontend/src/features/tags/TagModal.tsx +++ b/superset-frontend/src/features/tags/TagModal.tsx @@ -88,6 +88,14 @@ const TagModal: React.FC = ({ setSavedQueriesToTag([]); }; + const clearTagForm = () => { + setTagName(''); + setDescription(''); + setDashboardsToTag([]); + setChartsToTag([]); + setSavedQueriesToTag([]); + }; + useEffect(() => { const resourceMap: { [key: string]: TaggableResourceOption[] } = { [TaggableResources.Dashboard]: [], @@ -225,7 +233,9 @@ const TagModal: React.FC = ({ }) .then(({ json = {} }) => { refreshData(); + clearTagForm(); addSuccessToast(t('Tag updated')); + onHide(); }) .catch(err => { addDangerToast(err.message || 'Error Updating Tag'); @@ -241,24 +251,19 @@ const TagModal: React.FC = ({ }) .then(({ json = {} }) => { refreshData(); + clearTagForm(); addSuccessToast(t('Tag created')); + onHide(); }) .catch(err => addDangerToast(err.message || 'Error Creating Tag')); } - onHide(); }; return ( { - if (clearOnHide) { - setTagName(''); - setDescription(''); - setDashboardsToTag([]); - setChartsToTag([]); - setSavedQueriesToTag([]); - } + if (clearOnHide) clearTagForm(); onHide(); }} show={show}