fix(sdk): resolve immediately if already enabled with the same options

AztecProtocol · Feb 13, 2020 · 5e2fd22 · 5e2fd22
1 parent ccb3911
commit 5e2fd22
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/packages/extension/src/client/Aztec/ApiManager.js b/packages/extension/src/client/Aztec/ApiManager.js
@@ -1,3 +1,4 @@
+import isEqual from 'lodash/isEqual';
 import {
     getNetworkName,
 } from '~/utils/network';
@@ -10,6 +11,7 @@ export default class ApiManager {
     constructor() {
         this.eventListeners = new EventListeners(['profileChanged']);
         this.enableProfileChangeListener = null;
+        this.enabledOptions = null;
 
         Web3Service.bindProfileChange((changedType, newTypeValue) => {
             let objValue = newTypeValue || null;
@@ -65,6 +67,15 @@ export default class ApiManager {
         callback = null,
         setApis,
     ) => new Promise(async (resolve, reject) => {
+        if (isEqual(options, this.enabledOptions)) {
+            if (callback) {
+                callback(true, null);
+            }
+
+            resolve(true);
+            return;
+        }
+
         const {
             apiKey = '',
             providerUrl = '',
@@ -92,7 +103,7 @@ export default class ApiManager {
             }
 
             if (!shouldReject) {
-                resolve(!!error);
+                resolve(!error);
             } else if (!callback) {
                 reject(error);
             }
@@ -178,6 +189,8 @@ export default class ApiManager {
             callback(true, null);
         }
 
+        this.enabledOptions = options;
+
         doResolved({
             aztecAccount,
         });