Don't do online check when tab not visible - seems to lead to spuriou…

…s offline indicators.
Freegle · Dec 28, 2023 · 1848245 · 1848245
1 parent 81a0c09
commit 1848245
Showing 1 changed file with 23 additions and 21 deletions.
diff --git a/stores/misc.js b/stores/misc.js
@@ -94,35 +94,37 @@ export const useMiscStore = defineStore({
       }
     },
     async checkOnline() {
-      try {
-        const response = await this.fetchWithTimeout(
-          this.config.public.APIv2 + '/online',
-          null,
-          5000
-        )
+      if (this.visible) {
+        try {
+          const response = await this.fetchWithTimeout(
+            this.config.public.APIv2 + '/online',
+            null,
+            5000
+          )
 
-        if (response?.status === 200) {
-          const rsp = await response.json()
+          if (response?.status === 200) {
+            const rsp = await response.json()
 
-          if (!this.online) {
-            console.log('Back online')
-            this.online = rsp.online
+            if (!this.online) {
+              console.log('Back online')
+              this.online = rsp.online
+            }
+          } else {
+            // Null response happens when we time out
+            console.log('Offline', response)
+            this.online = false
+          }
+        } catch (e) {
+          if (this.online) {
+            console.log('Gone offline', e)
+            this.online = false
           }
-        } else {
-          // Null response happens when we time out
-          console.log('Offline', response)
-          this.online = false
-        }
-      } catch (e) {
-        if (this.online) {
-          console.log('Gone offline', e)
-          this.online = false
         }
       }
 
       this.onlineTimer = setTimeout(
         this.checkOnline,
-        this.visible ? 1000 : 30000
+        1000
       )
     },
     waitForOnline() {