fix-settings-update-flash (#1341)

Co-authored-by: Jiuling.Lei <[email protected]>

Author: ice201508

frontend/src/components/application_spaces/ApplicationSpaceDetail.vue

@@ -244,12 +244,13 @@
     }
   }
 
-  const fetchRepoDetail = async () => {
+  const fetchRepoDetail = async (isUpdate = false) => {
     if (isDataLoading.value) {
       return false
     }
-    
-    isDataLoading.value = true
+    if (!isUpdate) {
+      isDataLoading.value = true
+    }
     
     const url = `/${props.repoType}s/${props.namespace}/${props.repoName}`
 

frontend/src/components/application_spaces/ApplicationSpaceSettings.vue

@@ -828,7 +828,10 @@
           if (payload.hasOwnProperty('private')) {
             this.updateVisibility(payload.private)
           }
-          await this.fetchRepoDetail()
+          const shouldRefreshRepo = !(payload.hasOwnProperty('cluster_id') || payload.hasOwnProperty('resource_id'))
+          if (shouldRefreshRepo) {
+            await this.fetchRepoDetail(true)
+          }
         }
       },
 

frontend/src/components/codes/CodeSettings.vue

@@ -301,7 +301,7 @@
           if (payload.hasOwnProperty('private')) {
             this.updateVisibility(payload.private)
           }
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
         }
       },
 

frontend/src/components/datasets/DatasetSettings.vue

@@ -665,7 +665,7 @@
         }
         const { error } = await useFetchApi(updateReadmeEndpoint, option).put().json()
         if (!error.value) {
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage({ message: this.$t('all.updateSuccess'), type: 'success' })
         } else {
           ElMessage({ message: error.value.msg, type: 'warning' })
@@ -686,7 +686,7 @@
           ElMessage({ message: error.value.msg, type: 'warning' })
           throw new Error(error.value.msg)
         } else {
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage({ message: this.$t('all.updateSuccess'), type: 'success' })
         }
       },
@@ -725,7 +725,7 @@
         if (error.value) {
           ElMessage.warning(error.value.msg)
         } else {
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage.success(this.$t('datasets.edit.updateSuccess'))
         }
       },

frontend/src/components/endpoints/EndpointDetail.vue

@@ -141,11 +141,13 @@
     }
   }
 
-  const fetchRepoDetail = async () => {
+  const fetchRepoDetail = async (isUpdate = false) => {
     if (isDataLoading.value) {
       return false
     }
-    isDataLoading.value = true
+    if (!isUpdate) {
+      isDataLoading.value = true
+    }
     
     const url = `/models/${props.namespace}/${props.modelName}/run/${props.endpointId}`
 

frontend/src/components/endpoints/EndpointSettings.vue

@@ -491,14 +491,14 @@
     const { data, error } = await useFetchApi(endpointUpdateEndpoint, options)
       .put()
       .json()
-
+      
     if (error.value) {
       ElMessage({ message: error.value.msg, type: 'warning' })
     } else {
       if (payload.hasOwnProperty('private')) {
         repoDetailStore.updateVisibility(payload.private)
       }
-      fetchRepoDetail()
+      fetchRepoDetail(true)
       ElMessage({ message: data.value.msg, type: 'success' })
     }
   }

frontend/src/components/mcp/McpSettings.vue

@@ -539,7 +539,7 @@
       .json()
 
     if (data.value) {
-      fetchRepoDetail(false)
+      fetchRepoDetail(true)
       ElMessage({ message: t('all.updateSuccess'), type: 'success' })
     } else {
       ElMessage({ message: error.value.msg, type: 'warning' })
@@ -585,7 +585,7 @@
       if (payload.hasOwnProperty('private')) {
         updateVisibility(payload.private)
       }
-      fetchRepoDetail(false)
+      fetchRepoDetail(true)
       ElMessage({ message: 'Success', type: 'success' })
     }
   }

frontend/src/components/models/ModelSettings.vue

@@ -652,7 +652,7 @@
         }
         const { data, error } = await useFetchApi(updateReadmeEndpoint, option).put().json()
         if (data.value) {
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage({ message: this.$t('all.updateSuccess'), type: 'success' })
         } else {
           ElMessage({ message: error.value.msg, type: 'warning' })
@@ -670,7 +670,7 @@
         if (error.value) {
           ElMessage({ message: error.value.msg, type: 'warning' })
         } else {
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage({ message: this.$t('all.addSuccess'), type: 'success' })
         }
       },
@@ -712,7 +712,7 @@
           if (payload.hasOwnProperty('private')) {
             this.updateVisibility(payload.private)
           }
-          this.fetchRepoDetail()
+          this.fetchRepoDetail(true)
           ElMessage({ message: 'Success', type: 'success' })
         }
       }

frontend/src/components/shared/RepoDetail.vue

@@ -100,13 +100,15 @@
     )
   })
 
-  const fetchRepoDetail = async () => {
+  const fetchRepoDetail = async (isUpdate = false) => {
     if (isDataLoading.value) {
       return false
     }
     
-    isDataLoading.value = true
-    lastFetchTime.value = Date.now()
+    if (!isUpdate) {
+      isDataLoading.value = true
+      lastFetchTime.value = Date.now()
+    }
     
     const timestamp = Date.now()
     const url = `/${props.repoType}s/${props.namespace}/${props.repoName}?_t=${timestamp}`