Sugar client: fix Promise.all

Missed in #135: Promise.all should take an array as an argument. I've also added better error-logging to debug Sugar client errors.
zooniverse · May 6, 2021 · bfce5ae · bfce5ae
1 parent fc7f305
commit bfce5ae
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/sugar.js b/lib/sugar.js
@@ -22,7 +22,7 @@ if (typeof navigator !== 'undefined') {
   var sugarClient = new SugarClient();
 
   auth.listen('change', function() {
-    Promise.all(auth.checkCurrent(), auth.checkBearerToken())
+    Promise.all([auth.checkCurrent(), auth.checkBearerToken()])
       .then(function([ user, token ]) {
         if (user && token) {
           sugarClient.userId = user.id;
@@ -41,6 +41,7 @@ if (typeof navigator !== 'undefined') {
         }
       })
       .catch(function(e) {
+        console.error(e)
         throw new Error('Failed to checkCurrent auth from sugar client');
       });
   });