Merge pull request #356 from amplimindcc/355-fix-sse-throws-error-on-…

…timeout recursively reconnect SSE on timeout and clear on leaving page
amplimindcc · Jun 11, 2024 · 4ed0057 · 4ed0057
2 parents 81457d1 + 237ed5f
commit 4ed0057
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/src/pages/Admin/components/Submissions/Submissions.tsx b/src/pages/Admin/components/Submissions/Submissions.tsx
@@ -73,6 +73,14 @@ export default function Submissions() {
     const [rowData, setRowData] = useState<UserSubmissionTableElement[]>([]);
 
     useEffect(() => {
+        /**
+         * connect SSE to the backend to get the submission status changes in real time and update the table accordingly
+         * @author Timo Hauser
+         * @author David Linhardt
+         *
+         * @async
+         * @returns {void}
+         */
         const connect = async () => {
             try {
                 const res = await submission.heartbeat();
@@ -97,15 +105,19 @@ export default function Submissions() {
                             }
                         }
                     );
-                    sse.onerror = (event) => {
-                        toast.showToast(ToastType.ERROR, t('sseError', { event: event}));
+                    sse.onerror = () => {
+                        connect();
                     };
                 }
             } catch (e: unknown) {
                 toast.showToast(ToastType.ERROR, t('connectionError', { ns: 'main' }));
             }
         };
         connect();
+
+        return () => {
+            // clean up
+        };
     });
 
     /**