Related to #172, changing ordering in case of success to ensure that …

…references to state are correctly set before clearing promise reference and displaying error message if the async request fails
peter-murray · May 31, 2020 · eaf3c88 · eaf3c88
1 parent f982706
commit eaf3c88
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/api/Api.js b/lib/api/Api.js
@@ -165,11 +165,12 @@ module.exports = class Api {
       }
 
       self._syncPromise = self._syncPromise.then(data => {
-          self._syncPromise = null;
           self._state = new Cache(data);
           self._lastSyncTime = new Date().getTime();
+          self._syncPromise = null;
         })
-        .catch(() => {
+        .catch(err => {
+          console.error(`Failed to async load the bridge configuration data; ${err}`);
           self._syncPromise = null;
         });
     }