wormhole-foundation / wormhole-connect

Wormhole Connect brings all the functionality and utility of Wormhole right into your application and removes all of the complexity.
https://connect-in-style.wormhole.com/
Other
71 stars 48 forks source link

[QA] [Mainnet] Cannot connect to Sui Metamask Snap wallet when Phantom wallet is connected #1542

Open AigaD opened 5 months ago

AigaD commented 5 months ago

Preconditions: Wallet not connected Console is open

Environment: MacBook Pro, macOS Ventura 13.0.1, Chrome Version 120.0.6099.234

Tested from: https://wormhole-connect-mainnet.netlify.app/# and https://preview.portal-bridge-ui.pages.dev/c0216a7edf/#

Steps:

  1. Select Solana Network at sending From section
  2. Connect Phantom wallet
  3. Select Sui at sending To section
  4. Try to connect Sui Metamask Snap wallet
  5. Observe
  6. Refresh the page
  7. Select Sui at sending To section
  8. Connect Sui Metamask Snap wallet
  9. Select Solana Network at sending From section
  10. Connect Phantom wallet
  11. Observe

Actual result: Cannot connect Sui Metamask Snap wallet when Phantom wallet is connected first, console shows error: Uncaught (in promise) Error: Already connecting

Expected result: Sui Metamask Snap wallet is connected

Comments: Didn't reproduce this issue with Solflare wallet

Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/f62c6c10-d8af-4b18-8c4a-e531c35d8e19

kev1n-peters commented 5 months ago

I can connect Sui metamask snap if I connect it before connecting Phantom. So it appears that there's an issue with Phantom preventing the user from connecting Sui metamask snap.

AigaD commented 5 months ago

Reproduced from: https://portalbridge.com/#

Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/1df50346-5942-4192-aa13-0ceed61ddf3b

AigaD commented 4 months ago

Reproduced from: https://preview.portalbridge.com/7246284e5d/ Cannot connect to Sui Metamask Snap when Phantom wallet is connected first, error shown Error: Already connecting Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/ebe9a883-1cda-4993-b087-eebabd5d3107