Merge pull request #17 from perawallet/fix/connect-reject-error-type

fix: `connect` reject error type
perawallet · Jul 4, 2022 · 0377eb9 · 0377eb9
2 parents 7df147e + 2731ee8
commit 0377eb9
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -73,6 +73,9 @@ function App() {
       .reject((error) => {
         // You MUST handle the reject because once the user closes the modal, peraWallet.connect() promise will be rejected.
         // For the async/await syntax you MUST use try/catch
+        if (error?.data?.type !== "CONNECT_MODAL_CLOSED") {
+          // log the necessary errors
+        }
       });
   }
 

diff --git a/src/modal/peraWalletConnectModalUtils.tsx b/src/modal/peraWalletConnectModalUtils.tsx
@@ -54,9 +54,9 @@ function openPeraWalletConnectModal(rejectPromise?: (error: any) => void) {
         rejectPromise(
           new PeraWalletConnectError(
             {
-              type: "SESSION_CONNECT"
+              type: "CONNECT_MODAL_CLOSED"
             },
-            "The action canceled by the user."
+            "The modal has been closed by the user."
           )
         );
       }

diff --git a/src/util/PeraWalletConnectError.ts b/src/util/PeraWalletConnectError.ts
@@ -4,7 +4,8 @@ interface PeraWalletConnectErrorData {
     | "SESSION_DISCONNECT"
     | "SESSION_UPDATE"
     | "SESSION_CONNECT"
-    | "SESSION_RECONNECT";
+    | "SESSION_RECONNECT"
+    | "CONNECT_MODAL_CLOSED";
   detail?: any;
 }