fix: add missing closeModalOld for the old way of usage

argentlabs · Nov 6, 2024 · 6457371 · 6457371
1 parent e0669ce
commit 6457371
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/connectors/argent/argentMobile/modal/argentModal.ts b/src/connectors/argent/argentMobile/modal/argentModal.ts
@@ -172,7 +172,6 @@ class ArgentModal {
     })
   }
 
-  // TODO - SK-47 - remove this
   public closeModal(success?: boolean) {
     const modal = this.standaloneConnectorModal
     if (success) {
@@ -183,6 +182,19 @@ class ArgentModal {
     }
   }
 
+  // TODO - SK-47 - remove this
+  public closeModalOld(success?: boolean) {
+    if (success) {
+      this.overlay
+        ?.querySelector("iframe")
+        ?.contentWindow?.postMessage("argent-login.success", "*")
+      this.popupWindow?.postMessage("argent-login.success", "*")
+      this.closingTimeout = setTimeout(this.close, 3400)
+    } else {
+      this.close()
+    }
+  }
+
   private showModal(urls: Urls, modalWallet: ModalWalletExtended) {
     this.standaloneConnectorModal = new Modal({
       target: getModalTarget(),

diff --git a/src/connectors/argent/argentMobile/modal/starknet/adapter.ts b/src/connectors/argent/argentMobile/modal/starknet/adapter.ts
@@ -186,10 +186,10 @@ export class StarknetAdapter
       const chainId = this.formatChainId(this.chainId)
       argentModal.showApprovalModal(request)
       const response = await this.client.request({ topic, chainId, request })
-      argentModal.closeModal(true)
+      argentModal.closeModalOld(true)
       return response
     } catch (error) {
-      argentModal.closeModal()
+      argentModal.closeModalOld()
       if (error instanceof Error) {
         throw new Error(error.message)
       }