Merge pull request #43542 from dominictb/fix/43089

Expensify · Jun 23, 2024 · 5fb3cb6 · 5fb3cb6
2 parents 4dc1710 + ae03a7f
commit 5fb3cb6
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/src/pages/home/report/UserTypingEventListener.tsx b/src/pages/home/report/UserTypingEventListener.tsx
@@ -24,6 +24,23 @@ function UserTypingEventListener({report, lastVisitedPath}: UserTypingEventListe
     const reportID = report.reportID;
     const isFocused = useIsFocused();
     const route = useRoute<RouteProp<CentralPaneNavigatorParamList, typeof SCREENS.REPORT>>();
+
+    useEffect(
+        () => () => {
+            if (!didSubscribeToReportTypingEvents.current) {
+                return;
+            }
+
+            // unsubscribe from report typing events when the component unmounts
+            didSubscribeToReportTypingEvents.current = false;
+            InteractionManager.runAfterInteractions(() => {
+                Report.unsubscribeFromReportChannel(reportID);
+            });
+        },
+        // eslint-disable-next-line react-hooks/exhaustive-deps
+        [],
+    );
+
     useEffect(() => {
         // Ensures any optimistic report that is being created (ex: a thread report) gets created and initialized successfully before subscribing
         if (route?.params?.reportID !== reportID) {