[Enhancement]: Graceful offline support #590

When disconnected from the internet and Obsidian git still attempts to pull changes it throws some panicky error messages.

Check the connectivity before attempting to pull/push changes

Author: Bardoe Besselaar

src/main.ts

@@ -740,6 +740,11 @@ export default class ObsidianGit extends Plugin {
         return Platform.isDesktopApp;
     }
 
+    hasConnectivity() {
+        if (navigator.onLine) return true;
+        return (new Notice('No Connectivity'), false)
+    }
+
     async init(): Promise<void> {
         this.showNotices();
 
@@ -926,6 +931,7 @@ export default class ObsidianGit extends Plugin {
     ///Used for command
     async pullChangesFromRemote(): Promise<void> {
         if (!(await this.isAllInitialized())) return;
+        if (!(this.hasConnectivity())) return;
 
         const filesUpdated = await this.pull();
         this.setUpAutoBackup();
@@ -956,10 +962,13 @@ export default class ObsidianGit extends Plugin {
     ): Promise<void> {
         if (!(await this.isAllInitialized())) return;
 
+        const isConnected = this.hasConnectivity();
+
         if (
             this.settings.syncMethod == "reset" &&
             this.settings.pullBeforePush
         ) {
+            if (!isConnected) return;
             await this.pull();
         }
 
@@ -975,6 +984,7 @@ export default class ObsidianGit extends Plugin {
         if (!this.settings.disablePush) {
             // Prevent plugin to pull/push at every call of createBackup. Only if unpushed commits are present
             if (await this.gitManager.canPush()) {
+                if (!isConnected) return;
                 if (
                     this.settings.syncMethod != "reset" &&
                     this.settings.pullBeforePush
@@ -1166,9 +1176,9 @@ export default class ObsidianGit extends Plugin {
 
     async push(): Promise<boolean> {
         if (!(await this.isAllInitialized())) return false;
-        if (!(await this.remotesAreSet())) {
-            return false;
-        }
+        if (!(await this.remotesAreSet())) return false;
+        if (!(this.hasConnectivity())) return false;
+
         const hadConflict = this.localStorage.getConflict() === "true";
         if (this.gitManager instanceof SimpleGit)
             await this.mayDeleteConflictFile();