Merge pull request #2355 from nocodb/fix/sqlite-datetime

fix: show err message when input cannot be parsed

packages/nc-gui/components/project/spreadsheet/components/editableCell/DateTimePickerCell.vue

@@ -1,21 +1,27 @@
 <template>
   <div>
-    <v-datetime-picker
+    <div v-show="!showMessage">
-      ref="picker"
+      <v-datetime-picker
-      v-model="localState"
+        ref="picker"
-      class="caption xc-date-time-picker"
+        v-model="localState"
-      :text-field-props="{
+        class="caption xc-date-time-picker"
-        class:'caption mt-0 pt-0',
+        :text-field-props="{
-        flat:true,
+          class:'caption mt-0 pt-0',
-        solo:true,
+          flat:true,
-        dense:true,
+          solo:true,
-        hideDetails:true
+          dense:true,
-      }"
+          hideDetails:true
-      :time-picker-props="{
+        }"
-        format:'24hr'
+        :time-picker-props="{
-      }"
+          format:'24hr'
-      v-on="parentListeners"
+        }"
-    />
+        v-on="parentListeners"
+      />
+    </div>
+    <div v-show="showMessage" class="edit-warning" @dblclick="$refs.picker.display = true">
+      <!-- TODO: i18n -->
+      ERR: Couldn't parse {{ this.value }}
+    </div>
   </div>
 </template>
 
@@ -31,6 +37,9 @@ export default {
   props: {
     value: [String, Date, Number], ignoreFocus: Boolean
   },
+  data: () => ({
+    showMessage: false
+  }),
   computed: {
     isMysql() {
       return ['mysql', 'mysql2'].indexOf(this.$store.getters['project/GtrClientType'])
@@ -40,8 +49,13 @@ export default {
         if (!this.value) {
           return this.value
         }
-        return (/^\d+$/.test(this.value) ? dayjs(+this.value) : dayjs(this.value))
+        const d = (/^\d+$/.test(this.value) ? dayjs(+this.value) : dayjs(this.value))
-          .format('YYYY-MM-DD HH:mm')
+        if (d.isValid()) {
+          this.showMessage = false
+          return d.format('YYYY-MM-DD HH:mm')
+        } else {
+          this.showMessage = true
+        }
       },
       set(value) {
         if (this.isMysql) {
@@ -85,6 +99,12 @@ export default {
   padding-top: 0 !important;
   font-size: inherit !important;
 }
+
+.edit-warning {
+  padding: 10px;
+  text-align: left;
+  color: #E65100;
+}
 </style>
 <!--
 /**