refactor(Proof add): on Proof add success, redirect to dashboard. ref #…

…878
openfoodfacts · Oct 7, 2024 · ffe8c13 · ffe8c13
1 parent 8816898
commit ffe8c13
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/src/views/Home.vue b/src/views/Home.vue
@@ -29,13 +29,6 @@
     </v-col>
   </v-row>
 
-  <v-snackbar
-    v-model="proofSingleSuccessMessage"
-    color="success"
-    :timeout="2000"
-  >
-    {{ $t('ProofCreate.Success') }}
-  </v-snackbar>
   <v-snackbar
     v-model="settingsSuccessMessage"
     color="success"
@@ -67,7 +60,6 @@ export default {
       totalPriceCount: null,
       loading: false,
       // success messages
-      proofSingleSuccessMessage: false,
       settingsSuccessMessage: false,
     }
   },
@@ -85,9 +77,6 @@ export default {
     }
   },
   mounted() {
-    if (this.$route.query.proofSingleSuccess === 'true') {
-      this.proofSingleSuccessMessage = true
-    }
     if (this.$route.query.settingsSuccess === 'true') {
       this.settingsSuccessMessage = true
     }

diff --git a/src/views/ProofAddSingle.vue b/src/views/ProofAddSingle.vue
@@ -165,14 +165,17 @@ export default {
       this.loading = true
       api.updateProof(this.addProofSingleForm.proof_id, this.addProofSingleForm)
         .then(() => {
-          this.$router.push({ path: '/', query: { proofSingleSuccess: 'true' } })
+          this.done()
         })
         .catch(err => {
           this.$store.app.showError(err)
         })
         .finally(() => {
           this.loading = false
         })
+    },
+    done() {
+      this.$router.push({ path: '/dashboard', query: { proofSingleSuccess: 'true' } })
     }
   }
 }

diff --git a/src/views/UserDashboard.vue b/src/views/UserDashboard.vue
@@ -43,6 +43,13 @@
   >
     {{ $t('Common.Thanks') }}
   </v-snackbar>
+  <v-snackbar
+    v-model="proofSingleSuccessMessage"
+    color="success"
+    :timeout="2000"
+  >
+    {{ $t('ProofCreate.Success') }}
+  </v-snackbar>
 </template>
 
 <script>
@@ -68,6 +75,7 @@ export default {
       // success messages
       singleSuccessMessage: false,
       multipleSuccessMessage: false,
+      proofSingleSuccessMessage: false,
     }
   },
   computed: {
@@ -94,6 +102,9 @@ export default {
     if (this.$route.query.multipleSuccess === 'true') {
       this.multipleSuccessMessage = true
     }
+    if (this.$route.query.proofSingleSuccess === 'true') {
+      this.proofSingleSuccessMessage = true
+    }
   },
   methods: {
     getUser() {