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

useMaxSendAmount fails with CELO #2978

Open
MuckT opened this issue Oct 12, 2022 · 3 comments
Open

useMaxSendAmount fails with CELO #2978

MuckT opened this issue Oct 12, 2022 · 3 comments

Comments

@MuckT
Copy link
Collaborator

MuckT commented Oct 12, 2022

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
Copy link

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
Copy link
Collaborator Author

MuckT commented Oct 13, 2022

#2263 - Related Max Send issue.

@MuckT MuckT added Priority: P3 Minor and removed Priority: P2 Major labels Oct 13, 2022
@MuckT
Copy link
Collaborator Author

MuckT commented Oct 13, 2022

Downgrading priority as it is not 100% reproducible.

@MuckT MuckT assigned MuckT and unassigned MuckT Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants