Skip to content

Commit d1aad9f

Browse files
authored
Merge pull request #9 from blocknative/fix/auto-select-wallet
Fix `selectWallet` bug when using auto select functionality
2 parents 211d0b4 + 6c85d5d commit d1aad9f

File tree

1 file changed

+19
-20
lines changed

1 file changed

+19
-20
lines changed

src/views/WalletSelect.svelte

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -32,31 +32,30 @@
3232
const { wallets, heading, description } = module;
3333
const deviceWallets = wallets[mobileDevice ? "mobile" : "desktop"];
3434
35-
$: if ($app.autoSelectWallet) {
36-
const module = deviceWallets.find(m => m.name === $app.autoSelectWallet);
37-
handleWalletSelect(module);
38-
}
39-
4035
let primaryWallets;
4136
let secondaryWallets;
4237
43-
if (deviceWallets.find(wallet => wallet.preferred)) {
44-
// if preferred wallets, then split in to preferred and not preferred
45-
primaryWallets = deviceWallets.filter(wallet => wallet.preferred);
46-
secondaryWallets = deviceWallets.filter(wallet => !wallet.preferred);
38+
$: if ($app.autoSelectWallet) {
39+
const module = deviceWallets.find(m => m.name === $app.autoSelectWallet);
40+
handleWalletSelect(module);
4741
} else {
48-
// otherwise make the first 4 wallets preferred
49-
primaryWallets = deviceWallets.slice(0, 4);
50-
secondaryWallets = deviceWallets.length > 4 && deviceWallets.slice(4);
51-
}
42+
if (deviceWallets.find(wallet => wallet.preferred)) {
43+
// if preferred wallets, then split in to preferred and not preferred
44+
primaryWallets = deviceWallets.filter(wallet => wallet.preferred);
45+
secondaryWallets = deviceWallets.filter(wallet => !wallet.preferred);
46+
} else {
47+
// otherwise make the first 4 wallets preferred
48+
primaryWallets = deviceWallets.slice(0, 4);
49+
secondaryWallets = deviceWallets.length > 4 && deviceWallets.slice(4);
50+
}
5251
53-
// set the data to show in the modal if wallet hasn't been auto selected
54-
modalData = !$app.autoSelectWallet && {
55-
heading,
56-
description,
57-
primaryWallets,
58-
secondaryWallets
59-
};
52+
modalData = {
53+
heading,
54+
description,
55+
primaryWallets,
56+
secondaryWallets
57+
};
58+
}
6059
6160
function handleWalletSelect(module) {
6261
const { provider, interface: selectedWalletInterface } = module.wallet({

0 commit comments

Comments
 (0)