Check for game updates when viewing game page

alts · alts · commit 13761cf9fab5 · 2024-01-22T13:55:52.000-08:00
diff --git a/src/common/actions/index.ts b/src/common/actions/index.ts
@@ -692,6 +692,7 @@ export const actions = wireActions({
   checkForGameUpdate: action<{
     /** which cave to check for an update */
     caveId: string;
+    suppressNotification: boolean;
   }>(),
   gameUpdateCheckStatus: action<{
     /** whether we're currently checking */
diff --git a/src/main/reactors/context-menu/build-template.ts b/src/main/reactors/context-menu/build-template.ts
@@ -127,7 +127,10 @@ export function gameControls(store: Store, game: Game): MenuTemplate {
     if (!busy) {
       updateAndLocalItems.push({
         localizedLabel: ["grid.item.check_for_update"],
-        action: actions.checkForGameUpdate({ caveId: cave.id }),
+        action: actions.checkForGameUpdate({
+          caveId: cave.id,
+          suppressNotification: false,
+        }),
       });
     }
 
diff --git a/src/main/reactors/updater.ts b/src/main/reactors/updater.ts
@@ -114,7 +114,7 @@ export default function (watcher: Watcher) {
   });
 
   watcher.on(actions.checkForGameUpdate, async (store, action) => {
-    const { caveId } = action.payload;
+    const { caveId, suppressNotification } = action.payload;
     logger.info(`Looking for updates for cave ${caveId}`);
     const { cave } = await mcall(messages.FetchCave, { caveId });
 
@@ -145,7 +145,9 @@ export default function (watcher: Watcher) {
       }
     }
 
-    dispatchUpdateNotification(store, cave, res);
+    if (!suppressNotification) {
+      dispatchUpdateNotification(store, cave, res);
+    }
   });
 
   watcher.on(actions.snoozeCave, async (store, action) => {
diff --git a/src/renderer/modal-widgets/ManageCave.tsx b/src/renderer/modal-widgets/ManageCave.tsx
@@ -286,7 +286,10 @@ class ManageCave extends React.PureComponent<Props> {
     dispatch(
       actions.closeModal({
         wind: ambientWind(),
-        action: actions.checkForGameUpdate({ caveId }),
+        action: actions.checkForGameUpdate({
+          caveId,
+          suppressNotification: false,
+        }),
       })
     );
   };
diff --git a/src/renderer/pages/BrowserPage/BrowserContext/BrowserContextGame.tsx b/src/renderer/pages/BrowserPage/BrowserContext/BrowserContextGame.tsx
@@ -66,6 +66,20 @@ class BrowserContextGame extends React.PureComponent<Props> {
     );
   }
 
+  componentDidMount() {
+    const { status, dispatch } = this.props;
+    if (!status.operation && !status.update && status.cave) {
+      // this logic is inferred from the onClick handler
+      // defined in MainAction
+      dispatch(
+        actions.checkForGameUpdate({
+          caveId: status.cave.id,
+          suppressNotification: true,
+        })
+      );
+    }
+  }
+
   onContextMenu = (ev: React.MouseEvent<any>) => {
     ev.preventDefault();
     const { clientX, clientY } = ev;
