Merge pull request #604 from tradingstrategy-ai/connect-wallet-bug

Fix wallet connection bug
tradingstrategy-ai · Oct 11, 2023 · 81931c3 · 81931c3
2 parents fc1f134 + 7fb03bb
commit 81931c3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/src/lib/wallet/ConnectWallet.svelte b/src/lib/wallet/ConnectWallet.svelte
@@ -27,11 +27,11 @@
 		</div>
 	{:else}
 		<div class="wallet-options">
-			<WalletTile name="Browser Wallet" slug="browser-wallet" on:click={() => connectWallet('Injected')}>
+			<WalletTile name="Browser Wallet" slug="browser-wallet" on:click={() => connectWallet('injected')}>
 				Connect to your<br />
 				browser-based wallet
 			</WalletTile>
-			<WalletTile name="WalletConnect" slug="walletconnect" on:click={() => connectWallet('WalletConnect')}>
+			<WalletTile name="WalletConnect" slug="walletconnect" on:click={() => connectWallet('walletConnect')}>
 				Scan a QR code<br />
 				with your mobile wallet
 			</WalletTile>

diff --git a/src/lib/wallet/client.ts b/src/lib/wallet/client.ts
@@ -25,7 +25,7 @@ function getRpcUrl({ id }: Chain) {
 	return http && { http };
 }
 
-export type ConnectorType = 'Injected' | 'WalletConnect';
+export type ConnectorType = 'injected' | 'walletConnect';
 
 type CommonWallet = {
 	name: string;
@@ -88,11 +88,9 @@ function initWalletClient(connectors: Connector[]) {
 	initialized = true;
 }
 
-function connectWallet(type: ConnectorType, chainId: MaybeNumber) {
-	return connect({
-		chainId,
-		connector: connectors.find((c) => c.constructor.name === `${type}Connector`)!
-	});
+function connectWallet(type: ConnectorType, chainId: number | undefined) {
+	const connector = connectors.find((c) => c.id === type)!;
+	return connect({ chainId, connector });
 }
 
 export const wallet = { subscribe, connect: connectWallet, disconnect };