Discovered this issue while investigating #1006. In short, each login flow should set the walletType at the time of login, so that future delegations to the wallet (e.g., signing and sending) use the corresponding methods for a given wallet type.
However, it seems that it is not being set for Metamask / Brave Wallet (note in the below screenshot, it's being set for other wallet types as I clicked through them, but not MM).
The downstream effect of this is that sometimes when logging in with MM, it fails with the following error:
Discovered this issue while investigating #1006. In short, each login flow should set the
walletTypeat the time of login, so that future delegations to the wallet (e.g., signing and sending) use the corresponding methods for a given wallet type.However, it seems that it is not being set for Metamask / Brave Wallet (note in the below screenshot, it's being set for other wallet types as I clicked through them, but not MM).
The downstream effect of this is that sometimes when logging in with MM, it fails with the following error: