feat: undo/redo for cover image change

Signed-off-by: mertmit <mertmit99@gmail.com>

packages/nc-gui/components/smartsheet/toolbar/FieldsMenu.vue

@@ -146,6 +146,27 @@ const coverOptions = computed<SelectProps['options']>(() => {
   return [{ value: null, label: 'No Image' }, ...filterFields]
 })
 
+const updateCoverImage = async (val?: string | null) => {
+  if (
+    (activeView.value?.type === ViewTypes.GALLERY || activeView.value?.type === ViewTypes.KANBAN) &&
+    activeView.value?.id &&
+    activeView.value?.view
+  ) {
+    if (activeView.value?.type === ViewTypes.GALLERY) {
+      await $api.dbView.galleryUpdate(activeView.value?.id, {
+        fk_cover_image_col_id: val,
+      })
+      ;(activeView.value.view as GalleryType).fk_cover_image_col_id = val
+    } else if (activeView.value?.type === ViewTypes.KANBAN) {
+      await $api.dbView.kanbanUpdate(activeView.value?.id, {
+        fk_cover_image_col_id: val,
+      })
+      ;(activeView.value.view as KanbanType).fk_cover_image_col_id = val
+    }
+    reloadViewMetaHook?.trigger()
+  }
+}
+
 const coverImageColumnId = computed({
   get: () => {
     const fk_cover_image_col_id =
@@ -159,23 +180,19 @@ const coverImageColumnId = computed({
     return null
   },
   set: async (val) => {
-    if (
+    if (val !== coverImageColumnId.value) {
-      (activeView.value?.type === ViewTypes.GALLERY || activeView.value?.type === ViewTypes.KANBAN) &&
+      addUndo({
-      activeView.value?.id &&
+        undo: {
-      activeView.value?.view
+          fn: await updateCoverImage,
-    ) {
+          args: [coverImageColumnId.value],
-      if (activeView.value?.type === ViewTypes.GALLERY) {
+        },
-        await $api.dbView.galleryUpdate(activeView.value?.id, {
+        redo: {
-          fk_cover_image_col_id: val,
+          fn: await updateCoverImage,
-        })
+          args: [val],
-        ;(activeView.value.view as GalleryType).fk_cover_image_col_id = val
+        },
-      } else if (activeView.value?.type === ViewTypes.KANBAN) {
+      })
-        await $api.dbView.kanbanUpdate(activeView.value?.id, {
+
-          fk_cover_image_col_id: val,
+      await updateCoverImage(val)
-        })
-        ;(activeView.value.view as KanbanType).fk_cover_image_col_id = val
-      }
-      reloadViewMetaHook?.trigger()
     }
   },
 })

packages/nocodb-sdk/src/lib/Api.ts

@@ -902,7 +902,7 @@ export interface GalleryType {
   /** Model for Bool */
   deleted?: BoolType;
   /** Foreign Key to Cover Image Column */
-  fk_cover_image_col_id?: string;
+  fk_cover_image_col_id?: StringOrNullType;
   /** Foreign Key to Model */
   fk_model_id?: string;
   /** Foreign Key to View */
@@ -1229,7 +1229,7 @@ export interface KanbanType {
   /** View ID */
   fk_view_id?: IdType;
   /** Cover Image Column ID */
-  fk_cover_image_col_id?: IdType;
+  fk_cover_image_col_id?: StringOrNullType;
   /** Kanban Columns */
   columns?: KanbanColumnType[];
   /** Meta Info for Kanban */

packages/nocodb/src/schema/swagger.json

@@ -15629,7 +15629,7 @@
             "$ref": "#/components/schemas/Bool"
           },
           "fk_cover_image_col_id": {
-            "type": "string",
+            "$ref": "#/components/schemas/StringOrNull",
             "description": "Foreign Key to Cover Image Column"
           },
           "fk_model_id": {
@@ -16498,7 +16498,7 @@
             "description": "View ID"
           },
           "fk_cover_image_col_id": {
-            "$ref": "#/components/schemas/Id",
+            "$ref": "#/components/schemas/StringOrNull",
             "description": "Cover Image Column ID"
           },
           "columns": {