search

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
78 stars 57 forks source link

[QA] Comma allowed in the Amount field #2433

Open agodlevska opened 3 weeks ago

agodlevska commented 3 weeks ago

Describe the current behavior in detail

Checked on wormhole-connect-mainnet (connect v2) and wormhole-connect-testnet (connect v2) and macOS Sonoma 14.6.1, Google Chrome Version 127.0.6533.122 (Official Build) (x86_64).

Steps:

  1. Prepare to send 0.001 Sui from Moonbeam (MetaMask) to Fantom (MetaMask)
  2. Change the Amount field, input a value using a comma (,) as the decimal separator (e.g., 0,001, 0,01)
  3. Attempt to proceed with the transaction

Expected Result: Amount field should only allow the use of a period (.) as the decimal separator in the Amount field.

Actual Result: The application allows the use of a comma (,) in the Amount field, and the transaction can be initiated successfully, even though the format is not standard.

Attachment/Notes

https://github.com/user-attachments/assets/3da52037-b5b8-45ea-b172-875494e23372

Screenshot 2024-08-26 at 16 20 00
agodlevska commented 1 day ago

Checked on https://preview.portalbridge.com/b1f3c5115d/ and https://wormhole-connect-mainnet.netlify.app/.
When the MAX is clicked in the "Amount" field, the default input uses a comma instead of a period.

Attachment:

https://github.com/user-attachments/assets/2531ca69-0a59-408f-a701-ba8fbbb49b0f