-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Use currentBalance instead of availableBalance #5014
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
Ok. Thanks for making a clear distinction. I am working on one of the PR's which deal with this so I will update that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM
Updated, only tested on web since there's not actually a UI change. |
You stole my PR! lol jk. Updated your description to include that other issue and merged 👍 |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @stitesExpensify in version: 1.0.94-2 🚀
|
@madmax330 Hello! is this testable by us? we tried by adding a fully verified account but the payment option doesn't appear. |
No I put the internalQA label, but apparently it doesn't work the same here as it does for the web app. Sorry, we'll test this internally. |
Hey y'all, any update on the internal QA here? This is the last item on the checklist blocking a prod deploy. |
Looks like @Jag96 and @kevinksullivan are doing the internal QA for this. What's the latest fellas? |
No it's @joekaufmanexpensify and Kevin. Should be good to check off her soon |
🤦♂️ lol Sorry! |
Hence the importance of the Ionatan rule 😆 |
QA successful on staging - balance changed after paying an IOU. |
🚀 Deployed to production by @yuwenmemon in version: 1.0.95-1 🚀
|
Details
We're showing the wrong balance to the user. The
currentBalance
is the one we want the user to see.availableBalance
is used on the backend to decide wether or not to authorize transactions.cc @parasharrajat
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/173152
$ https://github.com/Expensify/Expensify/issues/173151
Tests
QA Steps
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android