Creating snapshot triggers infinite loop for checking status update

changelog/entries/unreleased/bug/3490_creating_snapshot_triggers_infinite_loop_for_checking_status.json

@@ -0,0 +1,8 @@
+{
+    "type": "bug",
+    "message": "Fix for creating snapshot triggers infinite loop for checking status update",
+    "domain": "database",
+    "issue_number": 3490,
+    "bullet_points": [],
+    "created_at": "2025-03-06"
+}

enterprise/web-frontend/modules/baserow_enterprise/locales/en.json

@@ -470,4 +470,5 @@
   "periodicDataSyncDeactivatedNotification": {
     "body": "{name} periodic data sync has been deactivated because it failed too many times consecutively."
   }
+  
 }

web-frontend/modules/core/components/export/ExportWorkspaceModal.vue

@@ -122,7 +122,6 @@ export default {
         const { data: job } = await ExportWorkspaceService(
           this.$client
         ).exportApplications(this.workspace.id, values)
-        this.job = job
         await this.createAndMonitorJob(job)
       } catch (error) {
         this.createLoading = false

web-frontend/modules/core/components/import/ImportWorkspaceModal.vue

@@ -284,7 +284,6 @@ export default {
         const { data: job } = await ImportWorkspaceService(
           this.$client
         ).triggerImport(this.workspace.id, this.resourceId)
-        this.job = job
         await this.createAndMonitorJob(job)
       } catch (error) {
         this.importing = false

web-frontend/modules/core/components/snapshots/SnapshotsModal.vue

@@ -151,7 +151,6 @@ export default {
           this.application.id,
           values
         )
-        this.job = job
         await this.createAndMonitorJob(job)
       } catch (error) {
         this.createLoading = false
@@ -167,6 +166,7 @@ export default {
       ) {
         this.snapshots.unshift(this.job.snapshot)
       }
+      this.$refs.form.v$.$reset()
     },
     onJobFailed() {
       this.createLoading = false

web-frontend/modules/core/mixins/job.js

@@ -91,8 +91,7 @@ export default {
      * @returns {Promise<void>}
      */
     async createAndMonitorJob(job) {
-      await this.$store.dispatch('job/create', job)
-      this.job = job
+      this.job = await this.$store.dispatch('job/create', job)
     },
     /**
      * Called from a component to cancel any job currently running.

web-frontend/modules/core/store/job.js

@@ -31,8 +31,17 @@ export const mutations = {
   SET_ITEMS(state, items) {
     state.items = items
   },
+  /**
+   * Adds new job to the store.
+   * Adding job might be triggered from various places i.e. backend POST response
+   * or from websocket message. In those cases we want to make sure we don't add
+   * the same job multiple times.
+   */
   ADD_ITEM(state, item) {
-    state.items.push(item)
+    const existingJobIndex = state.items.findIndex((job) => job.id === item.id)
+    if (existingJobIndex === -1) {
+      state.items.push(item)
+    }
   },
   UPDATE_ITEM(state, { id, values }) {
     const index = state.items.findIndex((item) => item.id === id)
@@ -180,6 +189,7 @@ export const actions = {
   create({ dispatch }, job) {
     dispatch('forceCreate', job)
     dispatch('tryScheduleNextUpdate')
+    return this.getters['job/get'](job.id)
   },
 
   /**

web-frontend/modules/database/locales/en.json

@@ -830,7 +830,8 @@
     "snapshot": "Snapshot",
     "importingState": "Creating",
     "reset": "Create another",
-    "noSnapshots": "No snapshots for this application yet."
+    "noSnapshots": "No snapshots for this application yet.",
+    "nameAlreadyExists": "A snapshot with this name already exists."
   },
   "deleteSnapshotModal": {
     "title": "Delete snapshot",