ton-blockchain / wallet-desktop

Standard wallet for Windows/MacOS/Linux
Other
220 stars 82 forks source link

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

Closed julialips closed 2 years ago

julialips commented 2 years 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

No response

EmelyanenkoK commented 2 years ago

Wrong repo. This issue will be accounted as if it was posted in https://github.com/toncenter/ton-wallet/issues

GetRekt69420 commented 4 months ago

use google chrome, not brave.. (this fixed the problem with me)