search

toncenter / ton-wallet

Web Wallet and Browser Extension for TON (The Open Network)
https://wallet.ton.org
GNU General Public License v3.0
160 stars 58 forks source link

"API request error" appears when sendings TONs with a comment more than 70 cyrillic symbols #184

Open julialips opened 1 year ago

julialips commented 1 year ago

Bug Type

Functional

Reproduction steps

Precondition: You have positive TON balance.

  1. Open the site https://wallet.ton.org/ and login into your account
  2. Click 'Send' button
  3. Enter a valid TON address into the 'Recipient wallet address' field
  4. Enter a valid TON amount
  5. Enter a text large than 70 cyrillic symbols into the 'Comment (optional)' field (e.g. "Марс — четвёртая по удалённости от Солнца и седьмая по размеру планета Солнечной системы; масса пл")
  6. Click the 'Send TON' button

Actual result

"API request error" appears. The payment is not being sent. изображение_2022-09-17_004715356

Expected result

Some ways:

  1. The error should not appear. I guess, the comment has a maximum valid length, so a user should know this. The field may cut the comment length to the maximum and not allow to enter more. Some symbol counter will be cool.
  2. Local validation, without sending API. If the length is more than maximum, show text error or highlight the field with red line.

Suggested Severity

Medium

Device

Additional Context

The issue was reported in wrong repo: https://github.com/ton-blockchain/wallet-desktop/issues/36#issue-1376542422