Merge pull request #1137 from nocodb/fix/csv-import

fix: some csv import issues

packages/nc-gui/components/project/spreadsheet/components/importExport/columnMappingModal.vue

@@ -7,7 +7,11 @@
         </v-card-title>
         <v-spacer />
         <v-btn
-          :disabled="!valid || requiredColumnValidationError"
+          :disabled="
+            !valid || 
+            (typeof requiredColumnValidationError === 'string' || requiredColumnValidationError) || 
+            (typeof noSelectedColumnError === 'string' || noSelectedColumnError)
+          "
           color="primary"
           large
           @click="$emit('import',mappings)"
@@ -21,6 +25,9 @@
       <div v-if="requiredColumnValidationError" class="error--text caption pa-2 text-center">
         {{ requiredColumnValidationError }}
       </div>
+      <div v-if="noSelectedColumnError" class="error--text caption pa-2 text-center">
+        {{ noSelectedColumnError }}
+      </div>
       <v-divider />
       <v-container fluid>
         <v-form ref="form" v-model="valid">
@@ -39,7 +46,7 @@
             <tbody>
               <tr v-for="(r,i) in mappings" :key="i">
                 <td>
-                  <v-checkbox v-model="r.enabled" class="mt-0" dense hide-details />
+                  <v-checkbox v-model="r.enabled" class="mt-0" dense hide-details @change="$refs.form.validate()"/>
                 </td>
                 <td class="caption" style="width:45%">
                   <div :title="r.sourceCn" style="">
@@ -117,6 +124,12 @@ export default {
         return `Following columns are required : ${missingRequiredColumns.map(c => c._cn).join(', ')}`
       }
       return false
+    },
+    noSelectedColumnError() {
+      if ((this.mappings || []).filter(v => v.enabled === true).length == 0) { 
+        return 'At least one column has to be selected' 
+      }
+      return false
     }
   },
   mounted() {
@@ -129,6 +142,11 @@ export default {
         return true
       }
 
+      // if it is not selected, then pass validation
+      if (!row.enabled) {
+        return true
+      }
+
       const v = this.meta && this.meta.columns.find(c => c._cn === _cn)
 
       if ((this.mappings || []).filter(v => v.destCn === _cn).length > 1) { return 'Duplicate mapping found, please remove one of the mapping' }
@@ -154,6 +172,7 @@ export default {
         }
         this.mappings.push(o)
       }
+      this.$nextTick(()=> this.$refs.form.validate())
     },
     getIcon(uidt) {
       return getUIDTIcon(uidt) || 'mdi-alpha-v-circle-outline'

packages/nc-gui/components/project/spreadsheet/components/moreActions.vue

@@ -289,6 +289,7 @@ export default {
         }
         this.columnMappingModal = false
         this.$store.commit('loader/MutClear')
+        this.$emit('reload')
         this.$toast.success('Successfully imported table data').goAway(3000)
       } catch (e) {
         this.$toast.error(e.message).goAway(3000)

packages/nc-gui/components/project/spreadsheet/rowsXcDataTable.vue

@@ -110,6 +110,7 @@
           :selected-view="selectedView"
           @showAdditionalFeatOverlay="showAdditionalFeatOverlay($event)"
           @webhook="showAdditionalFeatOverlay('webhooks')"
+          @reload="reload"
         />
       </div>
       <v-spacer class="h-100" @dblclick="debug=true" />