(feat) O3-3010: Disable triggering offline mode when offline disabled (…

…openmrs#962)
mccarthyaaron · Mar 28, 2024 · c94ac5b · c94ac5b
1 parent bdf68fe
commit c94ac5b
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/packages/shell/esm-app-shell/src/run.ts b/packages/shell/esm-app-shell/src/run.ts
@@ -186,8 +186,6 @@ function connectivityChanged() {
  * Runs the shell by importing the translations and starting single SPA.
  */
 function runShell() {
-  window.addEventListener('offline', connectivityChanged);
-  window.addEventListener('online', connectivityChanged);
   return setupI18n()
     .catch((err) => console.error(`Failed to initialize translations`, err))
     .then(() => start());
@@ -378,6 +376,11 @@ async function precacheImportMap() {
   });
 }
 
+function registerOfflineHandlers() {
+  window.addEventListener('offline', connectivityChanged);
+  window.addEventListener('online', connectivityChanged);
+}
+
 function setupOfflineCssClasses() {
   subscribeConnectivity(({ online }) => {
     const body = document.querySelector('body')!;
@@ -410,7 +413,8 @@ export function run(configUrls: Array<string>, offline: boolean) {
 
   return setupApps()
     .then(finishRegisteringAllApps)
-    .then(setupOfflineCssClasses)
+    .then(offline ? setupOfflineCssClasses : undefined)
+    .then(offline ? registerOfflineHandlers : undefined)
     .then(provideConfigs)
     .then(runShell)
     .catch(handleInitFailure)