fix: moved dispatch to correct position

Eyevinn · Apr 11, 2024 · acf3f9f · acf3f9f
1 parent 382f406
commit acf3f9f
Showing 1 changed file with 11 additions and 15 deletions.
diff --git a/src/components/production-line/use-rtc-connection.ts b/src/components/production-line/use-rtc-connection.ts
@@ -161,28 +161,14 @@ export const useRtcConnection = ({
   // Teardown
   useEffect(
     () => () => {
-      if (inputAudioStream !== "no-device") {
-        // TODO remove check-log
-        console.log("Dispatch of mediastream");
-        dispatch({
-          type: "CONNECTED_MEDIASTREAM",
-          payload: inputAudioStream,
-        });
-      }
       audioElements.forEach((el) => {
         console.log("Tearing down audio element");
         el.pause();
         // eslint-disable-next-line no-param-reassign
         el.srcObject = null;
-        // TODO remove check-log
-        console.log("Remove mediastream");
-        dispatch({
-          type: "CONNECTED_MEDIASTREAM",
-          payload: null,
-        });
       });
     },
-    [audioElements, dispatch, inputAudioStream]
+    [audioElements, dispatch]
   );
 
   useEffect(() => {
@@ -213,6 +199,11 @@ export const useRtcConnection = ({
         rtcPeerConnection,
         inputAudioStream,
       });
+
+      dispatch({
+        type: "CONNECTED_MEDIASTREAM",
+        payload: inputAudioStream,
+      });
     }
 
     const { teardown } = establishConnection({
@@ -232,6 +223,11 @@ export const useRtcConnection = ({
         onConnectionStateChange
       );
 
+      dispatch({
+        type: "CONNECTED_MEDIASTREAM",
+        payload: null,
+      });
+
       rtcPeerConnection.close();
     };
   }, [