fix: Close emotesChannel to avoid unexpected behavior in the browser (#…

…2406)
decentraland · Nov 8, 2022 · d994b6a · d994b6a · vercel · Nov 8, 2022
1 parent f51d80b
commit d994b6a
Showing 1 changed file with 19 additions and 15 deletions.
diff --git a/src/modules/editor/sagas.ts b/src/modules/editor/sagas.ts
@@ -501,23 +501,27 @@ function* handleSetWearablePreviewController() {
   if (controller) {
     const emotesChannel = createWearablePreviewChannel(controller)
 
-    while (true) {
-      try {
-        const event: string = yield take(emotesChannel)
-        switch (event) {
-          case PreviewEmoteEventType.ANIMATION_PLAY:
-            yield put(setEmotePlaying(true))
-            break
-          case PreviewEmoteEventType.ANIMATION_PAUSE:
-            yield put(setEmotePlaying(false))
-            break
-          case PreviewEmoteEventType.ANIMATION_END:
-            yield put(setEmotePlaying(false))
-            break
+    try {
+      while (true) {
+        try {
+          const event: string = yield take(emotesChannel)
+          switch (event) {
+            case PreviewEmoteEventType.ANIMATION_PLAY:
+              yield put(setEmotePlaying(true))
+              break
+            case PreviewEmoteEventType.ANIMATION_PAUSE:
+              yield put(setEmotePlaying(false))
+              break
+            case PreviewEmoteEventType.ANIMATION_END:
+              yield put(setEmotePlaying(false))
+              break
+          }
+        } catch (error) {
+          yield put(setEmotePlaying(false))
         }
-      } catch (error) {
-        yield put(setEmotePlaying(false))
       }
+    } finally {
+      emotesChannel.close()
     }
   }
 }