Merge pull request #192 from harmonherring/master

Fixes #191 - Use named listener functions
AlbericoD · Nov 26, 2024 · bd7838f · bd7838f
2 parents c7a6be8 + 08830e8
commit bd7838f
Showing 1 changed file with 17 additions and 13 deletions.
diff --git a/template/src/screens/background/components/Screen.tsx b/template/src/screens/background/components/Screen.tsx
@@ -52,22 +52,26 @@ const BackgroundWindow = () => {
     [desktop, ingame, start, stop]
   );
 
+  const onGameInfoUpdated = async (event: overwolf.games.GameInfoUpdatedEvent) => {
+    if (
+      event.runningChanged &&
+      event.gameInfo?.classId === HEARTHSTONE_CLASS_ID
+    ) {
+      startApp("onGameInfoUpdated");
+    }
+  };
+
+  const onAppLaunchTriggered = () => {
+    startApp("onAppLaunchTriggered");
+  };
+
   useEffect(() => {
     startApp("on initial load");
-    overwolf.games.onGameInfoUpdated.addListener(async (event) => {
-      if (
-        event.runningChanged &&
-        event.gameInfo?.classId === HEARTHSTONE_CLASS_ID
-      ) {
-        startApp("onGameInfoUpdated");
-      }
-    });
-    overwolf.extensions.onAppLaunchTriggered.addListener(() => {
-      startApp("onAppLaunchTriggered");
-    });
+    overwolf.games.onGameInfoUpdated.addListener(onGameInfoUpdated);
+    overwolf.extensions.onAppLaunchTriggered.addListener(onAppLaunchTriggered);
     return () => {
-      overwolf.games.onGameInfoUpdated.removeListener(() => {});
-      overwolf.extensions.onAppLaunchTriggered.removeListener(() => {});
+      overwolf.games.onGameInfoUpdated.removeListener(onGameInfoUpdated);
+      overwolf.extensions.onAppLaunchTriggered.removeListener(onAppLaunchTriggered);
     };
   }, [startApp]);