From c5d23fbcc7f3d27aa8ada5e144ec1e9c7e367623 Mon Sep 17 00:00:00 2001 From: "Michael S. Molina" <70410625+michael-s-molina@users.noreply.github.com> Date: Thu, 30 Sep 2021 14:52:48 -0300 Subject: [PATCH] fix: Unable to add dataset (#16925) --- .../src/views/CRUD/data/dataset/AddDatasetModal.tsx | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/superset-frontend/src/views/CRUD/data/dataset/AddDatasetModal.tsx b/superset-frontend/src/views/CRUD/data/dataset/AddDatasetModal.tsx index 527b92a07e..80debeffe7 100644 --- a/superset-frontend/src/views/CRUD/data/dataset/AddDatasetModal.tsx +++ b/superset-frontend/src/views/CRUD/data/dataset/AddDatasetModal.tsx @@ -16,7 +16,7 @@ * specific language governing permissions and limitations * under the License. */ -import React, { FunctionComponent, useState } from 'react'; +import React, { FunctionComponent, useState, useEffect } from 'react'; import { styled, t } from '@superset-ui/core'; import { useSingleViewResource } from 'src/views/CRUD/hooks'; import { isEmpty, isNil } from 'lodash'; @@ -60,9 +60,9 @@ const DatasetModal: FunctionComponent = ({ addDangerToast, ); - const setSaveButtonState = () => { + useEffect(() => { setDisableSave(isNil(datasourceId) || isEmpty(currentTableName)); - }; + }, [currentTableName, datasourceId]); const onDbChange = (db: { id: number; @@ -70,17 +70,14 @@ const DatasetModal: FunctionComponent = ({ backend: string; }) => { setDatasourceId(db.id); - setSaveButtonState(); }; const onSchemaChange = (schema?: string) => { setSchema(schema); - setSaveButtonState(); }; const onTableChange = (tableName: string) => { setTableName(tableName); - setSaveButtonState(); }; const onSave = () => {