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
81 stars 63 forks source link

[QA] Gas estimate on wormhole-connect differs from gas estimate in MetaMask notification/ Leap wallet - Cosmos Gateway route #1779

Closed tsadovska closed 3 weeks ago

tsadovska commented 7 months ago

Describe the current behavior in detail

Checked on https://wormhole-connect-mainnet.netlify.app/ [Mainnet] with macOS and Google Chrome.

Steps:

  1. Prepare to send 0.0002 CELO from Injective (Leap) to Avalanche (MetaMask)
  2. Click "Approve and proceed with transaction" button
  3. Check "Gas estimate" value in "Send to" section
  4. Click "Claim" button
  5. Check "Gas fee" in MetaMask notification

Actual result: "Gas estimate" on wormhole-connect is 0 AVAX ($0.00), but "Estimated fee" in MetaMask is ~0.0096641 AVAX $0.45.

Screenshot 2024-03-12 at 11 31 20

What is the expected/desired behavior

Gas estimate on wormhole-connect is similar to gas estimate in MetaMask notification.

AigaD commented 7 months ago

Tested from: https://preview.portalbridge.com/b1ba6e856a/ When sending asst from Kujira network Fee estimate on Portal is shown as 0 KUJI, on Leap wallet pop-up the gas fee is shown as 0.01 KUJI Attachment:

https://github.com/wormhole-foundation/wormhole-connect/assets/91047061/b89e6cd2-bcd0-43b9-9b15-8146bcf03483

tsadovska commented 7 months ago

Reproducible on https://preview.portalbridge.com/8228d00809/ [Mainnet] when sending 0.0001 USDC from Ethereum (MetaMask) to Avalanche (MetaMask).

tsadovska commented 6 months ago

Reproducible on https://deploy-preview-2053--wormhole-connect-mainnet.netlify.app/ [Mainnet] when sending 0.00001 USDT from Injective (Leap) to Ethereum (MetaMask).

AigaD commented 5 months ago

Tested from: Mainnet Gas estimate for ETH is shown as 0 ETH, but on Rabby pop-up shown as 0.00177964 ETH for transaction 0.000001 tBTC from Osmosis (Leap) to EThereum (Rabby) Attachment:

Screenshot 2024-05-16 at 11 50 00
tsadovska commented 2 months ago

Issue is reproducible on https://portalbridge.com/ with the scenario mentioned above, when sending WETH from WETH from Osmosis (Leap) to Ethereum (MetaMask).

Screenshot 2024-09-03 at 10 19 46

tsadovska commented 3 weeks ago

Closing the ticket, because gas estimates are no longer displayed on Confirm transaction screen after redesign.