Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rounding up asset values on bridge and send pages causes a mismatch in actual balance #21861

Closed
VolodLytvynenko opened this issue Dec 20, 2024 · 1 comment
Labels
wallet: Send all issues for the send page of the wallet wallet-core Issues for mobile wallet team
Milestone

Comments

@VolodLytvynenko
Copy link
Contributor

VolodLytvynenko commented Dec 20, 2024

better to take into work after merging #21595

Preconditions:

The user owns an asset with a balance rounded up for display purposes.
Example: The user's balance is 1.059 USDC, but the app shows 1.06 USDC as the maximum value.

actual value
Image

Steps to Reproduce:

  1. Navigate to the Bridge or Send setup page.
  2. Attempt to use the displayed max value (e.g., 1.06 USDC).

Actual Result:

The user believes they have 1.06 USDC due to rounding up and attempts to use this amount.
This causes a mismatch and shows ugly error
Image

Expected Result:

The maximum value displayed should be rounded down to the nearest usable precision (e.g., 1.05 USDC).
This ensures the user cannot enter a value exceeding their actual balance, avoiding transaction failures.

@VolodLytvynenko VolodLytvynenko added wallet-core Issues for mobile wallet team wallet: Send all issues for the send page of the wallet labels Dec 20, 2024
@VolodLytvynenko VolodLytvynenko added this to the 2.33.0 milestone Dec 20, 2024
@VolodLytvynenko
Copy link
Contributor Author

duplicate of #21389

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wallet: Send all issues for the send page of the wallet wallet-core Issues for mobile wallet team
Projects
Archived in project
Development

No branches or pull requests

1 participant