Merge pull request #1517 from rowyio/feat/auto-fill-table-id

ROWY-2110 fix broken table id prefill for table setup
rowyio · Jan 2, 2024 · 2b2e3b4 · 2b2e3b4
2 parents 34e9769 + c421071
commit 2b2e3b4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/components/TableSettingsDialog/TableName.tsx b/src/components/TableSettingsDialog/TableName.tsx
@@ -12,7 +12,7 @@ export interface ITableNameProps extends IShortTextComponentProps {
 
 export default function TableName({ watchedField, ...props }: ITableNameProps) {
   const {
-    field: { onChange, value },
+    field: { onChange },
     useFormMethods: { control },
     disabled,
   } = props;
@@ -25,12 +25,9 @@ export default function TableName({ watchedField, ...props }: ITableNameProps) {
       if (!touched && typeof watchedValue === "string" && !!watchedValue) {
         // if table name field is not touched, and watched value is valid, set table name to watched value
         onChange(startCase(watchedValue));
-      } else if (typeof value === "string") {
-        // otherwise if table name is valid, set watched value to table name
-        onChange(startCase(value.trim()));
       }
     }
-  }, [watchedValue, disabled, onChange, value]);
+  }, [watchedValue, disabled]);
 
   return <ShortTextComponent {...props} />;
 }
diff --git a/src/components/TableSettingsDialog/form.tsx b/src/components/TableSettingsDialog/form.tsx
@@ -213,7 +213,7 @@ export const tableSettings = (
       name: "name",
       label: "Table name",
       required: true,
-      watchedField: "name",
+      watchedField: "collection",
       assistiveText: "User-facing name for this table",
       autoFocus: true,
       gridCols: { xs: 12, sm: 6 },