فهرست منبع

Merge pull request #8166 from weseek/fix/132880-132998-show-fail-message-when-sync-fails-to-start

set sync status to syncExecuting before API request
Ryoji Shimizu 2 سال پیش
والد
کامیت
1457c000c2
1فایلهای تغییر یافته به همراه4 افزوده شده و 2 حذف شده
  1. 4 2
      apps/app/src/features/external-user-group/client/components/ExternalUserGroup/SyncExecution.tsx

+ 4 - 2
apps/app/src/features/external-user-group/client/components/ExternalUserGroup/SyncExecution.tsx

@@ -74,11 +74,13 @@ export const SyncExecution = ({
   const onSyncBtnClick = useCallback(async(e) => {
   const onSyncBtnClick = useCallback(async(e) => {
     e.preventDefault();
     e.preventDefault();
     try {
     try {
-      await requestSyncAPI(e);
-      setProgress({ total: 0, current: 0 });
+      // set sync status before requesting to API, so that setting to syncFailed does not get overwritten
       setSyncStatus(SyncStatus.syncExecuting);
       setSyncStatus(SyncStatus.syncExecuting);
+      setProgress({ total: 0, current: 0 });
+      await requestSyncAPI(e);
     }
     }
     catch (errs) {
     catch (errs) {
+      setSyncStatus(SyncStatus.syncFailed);
       toastError(t(errs[0]?.code));
       toastError(t(errs[0]?.code));
     }
     }
   }, [t, requestSyncAPI]);
   }, [t, requestSyncAPI]);