Merge pull request #40997 from Skakruk/40044-track-active-tab

Fix ActiveClientManager doesn't track the currently active tab well and needs improved
Expensify · May 10, 2024 · d7216ea · d7216ea
2 parents e7c29e6 + 0de4ae2
commit d7216ea
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/libs/ActiveClientManager/index.ts b/src/libs/ActiveClientManager/index.ts
@@ -45,6 +45,11 @@ Onyx.connect({
     },
 });
 
+const cleanUpClientId = () => {
+    activeClients = activeClients.filter((id) => id !== clientID);
+    ActiveClients.setActiveClients(activeClients);
+};
+
 /**
  * Add our client ID to the list of active IDs.
  * We want to ensure we have no duplicates and that the activeClient gets added at the end of the array (see isClientTheLeader)
@@ -53,6 +58,8 @@ const init: Init = () => {
     activeClients = activeClients.filter((id) => id !== clientID);
     activeClients.push(clientID);
     ActiveClients.setActiveClients(activeClients).then(resolveSavedSelfPromise);
+
+    window.addEventListener('beforeunload', cleanUpClientId);
 };
 
 /**