Fix WalletConnect transaction state issue about morphs-frontend HOT 9 CLOSED

@rcstanciu Exactly, it has to do with the HttpProvider for the WalletConnectConnector being mistakenly instantiated with chainId: 1, which works fine when you're using mainnet, but causes this bug on other networks. I tested on mainnet after launch and didn't see the bug -- are you seeing something different?

from morphs-frontend.

Closing as this only happens on non-mainnet networks.

from morphs-frontend.

@zakangelle Does it work properly on mainnet? What about testnets like Goerli?

from morphs-frontend.

@rcstanciu The Morphs app supports Rinkeby testnet and Ethereum mainnet. The bug surfaces when using Rinkeby -- it will still allow you to mint, but will not show a "Minting..." message (when transaction is pending) and "Success" message (when transaction is confirmed).

from morphs-frontend.

And on mainnet the bug does not happen at all? The promises return as they should, like with the regular InjectedConnector?

from morphs-frontend.

I didn't get to the point where I can test it on mainnet. I only tried with different testnets and local hardhat node.
I was actually considering replacing wagmi before I saw you referencing that issue, but now I think I will continue working with it 😃 I'm glad everything was fine in the end! Congrats on your launch!

from morphs-frontend.

@rcstanciu Appreciate it! And yeah, this is my first time using wagmi and it's been great outside of this one issue -- there's a few people working on fixing it, and I'll probably take a crack at it next week too if it's still lingering. If you have any other questions when integrating it feel free to hit me up!

from morphs-frontend.

Thanks to you, I looked at the source code of WalletConnect connector and found a workaround. If you set the chainId to the working network, in the options object when instantiating the WalletConnectConnector, it will work. I'm not sure if it will still work after some network changes but it's a start haha.

from morphs-frontend.

@rcstanciu Nice! I'll have to try that out. Curious if it works w/ switching networks after initial load.

from morphs-frontend.