fix: schedule only on successful sync

Author: isekovanic

src/channel_manager.ts

@@ -361,14 +361,10 @@ export class ChannelManager {
 
         if (!this.client.offlineDb.syncManager.syncStatus) {
           // FIXME: Should only schedule for true syncStatus, not otherwise
-          this.client.offlineDb.syncManager.scheduleSyncStatusChangeCallback(
-            async (syncStatus) => {
-              console.log('WILL TRY NOW VAL: ', syncStatus);
-              if (syncStatus) {
-                await queryChannelsRequest();
-              }
-            },
-          );
+          this.client.offlineDb.syncManager.scheduleSyncStatusChangeCallback(async () => {
+            console.log('WILL TRY NOW RUNNING SCHEDULED NOW !');
+            await queryChannelsRequest();
+          });
           return;
         }
         // await queryChannelsRequest();

src/offline_support_api.ts

@@ -881,12 +881,12 @@ export class OfflineDBSyncManager {
     this.syncStatus = status;
     this.syncStatusListeners.forEach((l) => l(status));
 
-    console.log('WILL TRY LISTENERS', this.scheduledSyncStatusCallbacks.length);
-    const promises = this.scheduledSyncStatusCallbacks.map((cb) => cb(status));
-    await Promise.all(promises);
-    console.log('DONE LISTENERS !');
+    if (status) {
+      const promises = this.scheduledSyncStatusCallbacks.map((cb) => cb(status));
+      await Promise.all(promises);
 
-    this.scheduledSyncStatusCallbacks = [];
+      this.scheduledSyncStatusCallbacks = [];
+    }
   };
 
   private handleEventToSyncDB = async (event: Event, flush?: boolean) => {