search

valora-inc / wallet

The official repository for the Valora mobile cryptocurrency wallet.
https://valoraapp.com
Apache License 2.0
170 stars 80 forks source link

`useMaxSendAmount` fails with CELO #2978

Open MuckT opened 1 year ago

MuckT commented 1 year ago

Current behavior

Attempting to send the max amount of CELO fails to send. Does not occur with cUSD, cEUR or cREAL.

Desired behavior

Max send CELO should be produce a valid transaction.

Steps to Reproduce

Prerequisite: Wallet has Celo and no cEUR, cUSD or cREAL.

  1. Navigate to Send
  2. Select CELO
  3. Select Amount in crypto
  4. Select Max
  5. Attempt to send
  6. Observe transaction reverted - example reverted transaction

Versions

Valora 1.24.x and likely earlier.

ValoraQA commented 1 year ago

Hey @MuckT we have verified the above issue on Latest Android Internal Release build and Latest iOS Test Flight Release build V 1.42.0 and observed that issue is not replicated

Observation : We are able to send Max CELO with crypto and without crypto(without swapping places) and transaction are successful

Attachments : celo.mp4 Celo crypto.mp4

MuckT commented 1 year ago

2263 - Related Max Send issue.

MuckT commented 1 year ago

Downgrading priority as it is not 100% reproducible.