Fix signup and chain id check (#595)

cartridge-gg · Aug 19, 2024 · 980fe0b · 980fe0b
1 parent 11a5a90
commit 980fe0b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/packages/keychain/src/components/connect/Signup.tsx b/packages/keychain/src/components/connect/Signup.tsx
@@ -85,7 +85,7 @@ export function Signup({
       return;
     }
 
-    doSignup(decodeURIComponent(usernameField.value)).catch((e) => {
+    doSignup(usernameField.value).catch((e) => {
       // Backward compat with iframes without this permission-policy
       if (e.message.includes("publickey-credentials-create")) {
         doPopup();
@@ -125,7 +125,6 @@ export function Signup({
             },
           } = data;
 
-          console.log(origin);
           const controller = new Controller({
             appId: origin,
             chainId,

diff --git a/packages/keychain/src/hooks/deploy.ts b/packages/keychain/src/hooks/deploy.ts
@@ -40,6 +40,10 @@ export const useDeploy = (): DeployInterface => {
     async (username: string) => {
       if (isDeployed) return;
 
+      if (!chainId) {
+        throw new Error("Chain ID not set");
+      }
+
       if (chainId === constants.StarknetChainId.SN_MAIN)
         throw new Error("Mainnet not supported");