Move setUploading call into xhr events

regalijan · Mar 26, 2024 · c9c4cd5 · c9c4cd5
1 parent 0edbe2a
commit c9c4cd5
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/app/routes/report.tsx b/app/routes/report.tsx
@@ -184,16 +184,21 @@ export default function () {
       xhr.upload.addEventListener("progress", increaseProgress);
       xhr.upload.onabort = () => {
         shouldRecall = true;
+        setUploading(false);
+        setFileProgress(0);
       };
       xhr.upload.onerror = () => {
         shouldRecall = true;
+        setUploading(false);
+        setFileProgress(0);
+      };
+      xhr.upload.onloadend = () => {
+        if (i === upload_urls.length - 1) setUploading(false);
       };
 
       xhr.send(files[i]);
     }
 
-    setUploading(false);
-
     if (shouldRecall) {
       setLoading(false);
       await fetch("/api/reports/recall", {
@@ -309,7 +314,7 @@ export default function () {
             Submit
           </Button>
           <CircularProgress
-            display={uploading ? "" : "none"}
+            display={uploading ? undefined : "none"}
             value={fileProgress}
           >
             <CircularProgressLabel>{fileProgress}%</CircularProgressLabel>