Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WalletConnect is broken on Testnet #636

Closed
Viterbo opened this issue Oct 13, 2023 · 1 comment · Fixed by #642
Closed

WalletConnect is broken on Testnet #636

Viterbo opened this issue Oct 13, 2023 · 1 comment · Fixed by #642
Assignees
Labels
🐛 Bug Something isn't working

Comments

@Viterbo
Copy link
Collaborator

Viterbo commented Oct 13, 2023

Describe the bug
You can't sign any transaction using WalletConnect + Metamask in testnet. This bug affects all transactions but only this one is behaving differently because the implementation differs from the rest.

To Reproduce
Steps to reproduce the behavior:

  1. Open your Metamask and set Telos Mainnet as current chain
  2. Go to https://wallet-dev.telos.net
  3. Login using WalletConnect + Metamask
  4. copy your address
  5. open your console (dev tool - F12)
  6. goto Send section, paste your address, enter any amount, and wait for the error
  7. the loading never stops

Expected behavior
Not to have an error, the loading ends.

Screenshots
image

@Viterbo
Copy link
Collaborator Author

Viterbo commented Oct 16, 2023

The problem comes at the moment of populating this logalStogare variable wagmi.store as you can see in the screenshot it holds the wrong network data { id: 40, ... }
Captura desde 2023-10-16 19-06-41

This is causing every time Wagmi tries to call internally getNetwork(), it gets the wrong network and that causes an error later on the same call.

@Viterbo Viterbo self-assigned this Oct 16, 2023
@Viterbo Viterbo linked a pull request Oct 16, 2023 that will close this issue
10 tasks
@Viterbo Viterbo mentioned this issue Oct 16, 2023
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant