feat: skip fetching if offline (#2586)

Signed-off-by: Adam Setch <adam.setch@outlook.com>

Author: setchy

src/renderer/hooks/timers/useInactivityTimer.ts

@@ -1,5 +1,7 @@
 import { useCallback, useEffect, useRef } from 'react';
 
+import { isOnline } from '../../utils/network';
+
 const events = ['mousedown', 'keypress', 'click'];
 
 /**
@@ -23,8 +25,10 @@ export const useInactivityTimer = (callback: () => void, delay: number) => {
 
     if (delay !== null && savedCallback.current) {
       timeoutRef.current = setTimeout(() => {
-        // Fire callback once inactivity threshold reached
-        savedCallback.current();
+        // Fire callback once inactivity threshold reached if online
+        if (savedCallback.current && isOnline()) {
+          savedCallback.current();
+        }
 
         // Schedule next run while still inactive
         resetTimer();

src/renderer/hooks/timers/useIntervalTimer.ts

@@ -1,5 +1,7 @@
 import { useEffect, useRef } from 'react';
 
+import { isOnline } from '../../utils/network';
+
 /**
  * Hook that triggers a callback after a specified period of time.
  *
@@ -16,7 +18,9 @@ export const useIntervalTimer = (callback: () => void, delay: number) => {
   // Set up the interval.
   useEffect(() => {
     function tick() {
-      savedCallback.current();
+      if (savedCallback.current && isOnline()) {
+        savedCallback.current();
+      }
     }
 
     if (delay !== null) {

src/renderer/utils/network.ts

@@ -0,0 +1,6 @@
+// Simple utility to centralize online status checks
+export const isOnline = (): boolean => {
+  return typeof navigator !== 'undefined' ? navigator.onLine : true;
+};
+
+export default isOnline;