Merge pull request #118 from argentlabs/SK-9

fix: wc params for argent mobile
argentlabs · Aug 6, 2024 · b3a413f · b3a413f
2 parents da2a1e2 + 4e4a988
commit b3a413f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 16 deletions.
diff --git a/src/connectors/argentMobile/modal/argentModal.ts b/src/connectors/argentMobile/modal/argentModal.ts
@@ -72,21 +72,8 @@ class ArgentModal {
     const wcParam = encodeURIComponent(wcUri)
     const href = encodeURIComponent(window.location.href)
 
-    /* 
-    // KEEPING IN CASE NEEDS TO BE REVERTED
-    const desktopWcParam = encodeURIComponent(
-      `${this.mobileUrl}app/wc?uri=${wcParam}`,
-
-    this.showModal({
-      desktop: `${this.bridgeUrl}?wc=${desktopWcParam}&device=desktop`,
-      ios: `${this.mobileUrl}app/wc?uri=${wcParam}&href=${href}&device=mobile`,
-      android: `${this.mobileUrl}app/wc?uri=${wcParam}&href=${href}&device=mobile`,
-    })
-
-    ) */
-
     this.showModal({
-      desktop: `${this.bridgeUrl}?wc=${wcParam}&device=desktop`,
+      desktop: `${this.bridgeUrl}?wc=${wcParam}&href=${href}&device=desktop`,
       ios: `${this.mobileUrl}app/wc?uri=${wcParam}&href=${href}&device=mobile`,
       android: `${this.mobileUrl}app/wc?uri=${wcParam}&href=${href}&device=mobile`,
     })
@@ -109,7 +96,7 @@ class ArgentModal {
     this should be ignored and not considered valid as it's only used for automatically redirecting the users to approve or reject a signing request.
     */
     this.showModal({
-      desktop: `${this.bridgeUrl}?action=sign&device=desktop`,
+      desktop: `${this.bridgeUrl}?action=sign&device=desktop&href=${href}`,
       ios: `${this.mobileUrl}app/wc/request?href=${href}&device=mobile`,
       android: `${this.mobileUrl}app/wc/request?href=${href}&device=mobile`,
     })

diff --git a/src/connectors/argentMobile/modal/login.ts b/src/connectors/argentMobile/modal/login.ts
@@ -129,7 +129,7 @@ const getBridgeUrl = (chainId: unknown) => {
     String(chainId).startsWith(Network.SN_SEPOLIA) ||
     chainIdNumber === 11155111 // sepolia numeric value
   ) {
-    return "https://login.hydrogen.argent47.net"
+    return "https://mobile-login.hydrogen.argent47.net"
   }
 
   if (