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

Bug: doing any TX while on a wc2 connection opens metamask page #431

Open
1 task done
L03TJ3 opened this issue Jun 16, 2023 · 5 comments
Open
1 task done

Bug: doing any TX while on a wc2 connection opens metamask page #431

L03TJ3 opened this issue Jun 16, 2023 · 5 comments
Assignees
Labels
3rd-party-bug bug Something isn't working

Comments

@L03TJ3
Copy link
Collaborator

L03TJ3 commented Jun 16, 2023

Bug Description
User is navigated from gooddapp to the download page for metamask whenever doing any tx while using wallet-connectv2

Steps to Reproduce (taken from thread on onboard issue)

  1. use onboard connect modal (latest)
  2. click on wallet-connect (v2)
  3. click on one of the shown deeplink-wallets
  4. DONT CONNECT
  5. click back, and use QR scanning or copy URI to connect an alternative wallet
  6. nce connected, attempt to do a transaction
  7. redirect happens on dapp
    *. can verify here that the page that opens is the link set in storage on walletconnect_deeplink_choice

Expected Behavior
This should not happen

  • - remove all wallets from desktop list which don't have a desktop app

Additional context
As described in below 3rd party threads, the issue lies with the local storage item walletconnect_deeplink_choice
this never gets cleared properly and can be mismatched with an actual wallet being connected causing an incorrect redirect to a deeplink uri

@L03TJ3 L03TJ3 added the bug Something isn't working label Jun 16, 2023
@L03TJ3 L03TJ3 self-assigned this Jun 16, 2023
@L03TJ3
Copy link
Collaborator Author

L03TJ3 commented Jun 16, 2023

WalletConnect/walletconnect-monorepo#2303
wallet-connect issue thread

open one on web3-onboard: blocknative/web3-onboard#1780
since the behavior with the modal used on wc2 demo app is different then when using onboard

@L03TJ3
Copy link
Collaborator Author

L03TJ3 commented Jun 16, 2023

f699647
hotfix for now. will look after weekend to handle it more properly

@sirpy
Copy link
Contributor

sirpy commented Jun 18, 2023

@L03TJ3
How is it possible that we have missed this issue so far?
It looks like something broke, an upgraded package?

@L03TJ3
Copy link
Collaborator Author

L03TJ3 commented Jun 19, 2023

There are a couple of reasons why this could have been missed.
the most logical/easy one is that with my own testing of wallet-connect, I clear the cache pretty consistently so this particular issue does not come into existence.

@L03TJ3
Copy link
Collaborator Author

L03TJ3 commented Jun 30, 2023

Now its run before connecting / after disconnecting: da8049d

It could still happen when user navigates within the modal. issue open here: WalletConnect/modal#15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rd-party-bug bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants
@sirpy @L03TJ3 and others