aragon / app

Human-centered DAO infrastructure
https://app.aragon.org
GNU General Public License v3.0
46 stars 45 forks source link

Bug: Wrong Network Notice #1249

Closed jonathanstanley closed 7 months ago

jonathanstanley commented 9 months ago

Describe the bug cannot create dao due to "Wrong Network" error

To Reproduce (please complete the following information)

  1. web3 sign-in using trust wallet / wallet connect (with support for ethereum, arbitrum, and polygon).
  2. configure dao to use arbitrum chain
  3. click create dao
  4. receive chain network error Wrong network. The action can't be executed because you are in the wrong network. Change to the Arbitrum on your wallet and try again.

Current behavior throws error. dao creation is blocked for arbitrum

Expected behavior should not throw error

System (please complete the following information):

Additional context

jonathanstanley commented 9 months ago

I think this may be related: https://github.com/aragon/app/blob/e8b6a4654e2ed3891a15bc7dc5c2654a22c5e4ea/src/hooks/useWallet.tsx#L52-L56

aragon is using wagmi useNetwork however wagmi has removed that tool: https://wagmi.sh/react/guides/migrate-from-v1-to-v2#removed-usenetwork-hook

and perhaps use the plural const { chains } = useConfig()

Barukimang commented 8 months ago

Just seeing this now @jonathanstanley, good to know about this. One of our Devs is working on updating Wagmi, will check whether this issue has resolved by next week.

In some apps you cannot select the network anymore (like Rainbow) and depend on the network switch tool you mention. Not sure for Trust though, but adding this tool is on our timeline as well.

Thanks for the feedback!

Barukimang commented 7 months ago

Hi guys! We have updated the WAGMI package. Could you check whether wallet connections are working well for you now?

jonathanstanley commented 7 months ago

Thanks. FWIW, I still get the same error as I described earlier

Screenshot 2024-04-17 at 10 37 04 PM
Barukimang commented 7 months ago

Gm @jonathanstanley Seems like some other wallet connect issues have been fixed, but not yet this one where we need to implement a network switcher in our App for wallets that now depend on it. It's on our TD list though.