Unique column selection popup changed to fit on the screen. Improvements to saving data groupings when validation errors are present.

Author: piotrczarnas

dqops/src/main/frontend/src/components/ColumnsRecordDialog/ColumnsRecordDialog.tsx

@@ -104,7 +104,7 @@ const ColumnsRecordDialog = ({
         style={{ width: '200px !important', minWidth: '25%', maxWidth: '25%' }}
         className="pt-6"
       >
-        <div className="w-full h-full overflow-hidden">
+        <div className="w-full h-full overflow-x-hidden max-h-[50vh] overflow-y-auto" >
           <table
             className="table-fixed text-black text-sm w-full"
             style={{ tableLayout: 'fixed' }}

dqops/src/main/frontend/src/components/Connection/TableView/DataGroupingConfigurationEditView.tsx

@@ -62,6 +62,30 @@ const DataGroupingConfigurationEditView = ({
 
   const onUpdate = async () => {
     try {
+      if (name === '') {
+        return;
+      }
+
+      if (!dataGroupingConfiguration) {
+        setError('Grouping Configuration is Required');
+        return;
+      }
+      const errors: Errors = {};
+
+      Object.entries(dataGroupingConfiguration).forEach(([level, item]) => {
+        if (
+          item.source === DataGroupingDimensionSpecSourceEnum.tag &&
+          !item.tag
+        ) {
+          errors[level] = 'Tag is Required';
+        }
+      });
+
+      if (Object.values(errors).length) {
+        setLevelErrors(errors);
+        return;
+      }
+
       setIsUpdating(true);
       if (selectedGroupingConfiguration) {
         await DataGroupingConfigurationsApi.updateTableGroupingConfiguration(
@@ -75,30 +99,6 @@ const DataGroupingConfigurationEditView = ({
           }
         );
       } else {
-        if (name === '') {
-          return;
-        }
-
-        if (!dataGroupingConfiguration) {
-          setError('Grouping Configuration is Required');
-          return;
-        }
-        const errors: Errors = {};
-
-        Object.entries(dataGroupingConfiguration).forEach(([level, item]) => {
-          if (
-            item.source === DataGroupingDimensionSpecSourceEnum.tag &&
-            !item.tag
-          ) {
-            errors[level] = 'Tag is Required';
-          }
-        });
-
-        if (Object.values(errors).length) {
-          setLevelErrors(errors);
-          return;
-        }
-
         await DataGroupingConfigurationsApi.createTableGroupingConfiguration(
           connection,
           schema,
@@ -109,6 +109,7 @@ const DataGroupingConfigurationEditView = ({
           }
         );
       }
+
       setIsUpdated(false);
       getGroupingConfigurations();
       onBack();