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
76 stars 56 forks source link

[QA] Notification to change network is not shown on mobile #1358

Open tsadovska opened 8 months ago

tsadovska commented 8 months ago

Describe the current behavior in detail

Builds - Tested NOK:

Checked on macOS 12.7.1 and Google Chrome Version 120.0.6099.109 (Official Build) (x86_64).

Preconditions: Polygon Mumbai network is selected on Metamask app

Steps:

  1. Connect Metamask wallets through WalletConnect in From and To sections
  2. Prepare to send 2.00201 USDC from Ethereum to Polygon
  3. Click "Approve and proceed with transaction button"

Actual result: Notification to change network from Polygon Mumbai to Ethereum is not shown.

Error is displayed in Console: {code: 5000, message: '{"code":-32602,"message":"May not specify default MetaMask chain."}'}

image

What is the expected/desired behavior

Notification to change network from Polygon Mumbai to Ethereum is shown.

tsadovska commented 7 months ago

Issue is also reproducible on https://wormhole-connect.netlify.app/ [Testnet]. Steps:

  1. Moonbeam network is selected in Metamask app on mobile
  2. Connect Metamask wallet through WalletConnect in From and To sections
  3. Prepare to send 0.001 ETH from Goerli to Mumbai
  4. Click "Approve and proceed with transaction" button
  5. Check if notification to switch network is shown on mobile

Actual result:

tsadovska commented 4 months ago

Reproducible on https://testnet.portalbridge.com/ed4e22522f/ [Testnet] (Portal Bridge: 0.2.12, Wormhole Connect: 0.3.4-beta.4-development) with following scenario:

  1. Arbitrum Sepolia network is selected in MetaMask app on mobile
  2. Open https://testnet.portalbridge.com/ed4e22522f/ on Desktop
  3. Send 0.0001 W from Solana (Phantom) to Sepolia (MetaMask through WalletConnect)
  4. Click "Approve and proceed with transaction" button
  5. Click "Add Ws to your wallet" link
  6. Check if notification to change network is shown on mobile

Actual result:

Screenshot 2024-04-17 at 12 34 21

tsadovska commented 1 month ago

Issue is also reproducible on https://portalbridge.com/ [Mainnet] when sending 0.00001 wstETH from BSC (MetaMask through WalletConnect) to Ethereum (MetaMask through WalletConnect). Notification to change network from BSC to Ethereum does not appear in MetaMask mobile app.

Errors appear in Console after user clicks on "Add wstETH to your wallet":

{context: 'client'} '{"code":-32603,"message":"This token is not supported on this network","data":{"originalError":{}}}'

'{"code":-32602,"message":"May not specify default MetaMask chain."}'

Uncaught (in promise) {code: 5000, message: '{"code":-32602,"message":"May not specify default MetaMask chain."}'}

image