IOU - Pay button is visible for a moment after cancel requested amount.

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Issue found when executing PR #12488

Action Performed:

1. Launch the app
2. Log in with any account
3. Send a money request in USD while being online, e.g. $100.
4. Send a second money request in a different currency, e.g. 100 AED.
5. Go offline, cancel the request in USD
6. Go online.
7. Cancel the request in AED

Expected Result:

Pay button is not visible for a moment after cancel requested amount.

Actual Result:

Pay button is visible for a moment after cancel requested amount.

Workaround:

Unknown

Platform:

Where is this issue occurring?

• iOS
• Android

Version Number: 1.2.28.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Screen_Recording_20221115-222207_New.Expensify.1.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[melvin-bot]

Triggered auto assignment to @sophiepintoraetz (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[melvin-bot]

Triggered auto assignment to @techievivek (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[sophiepintoraetz]

Though cc @youssef-lr as you're in the linked PR this stemmed from.

[youssef-lr]

Thanks for the ping @sophiepintoraetz - I've reverted my PR locally and I can still reproduce this, so I don't think this is a regression bug, but just something we didn't notice before. The steps to reproduce this is simply to request money while online using a different currency than the report's, then cancel the request. I may have a fix for this one, looking into it today.

[youssef-lr]

I think we may need to cancel updating the IOU if the IOU's currency is different than the report's currency all together, regardless of whether or not the user is offline.

The issue here is that say we request 100 MAD from a user, and the default report's currency is USD, that's about $11. If we cancel the 100 MAD, in the code this amount is not converted to USD, so we're subtracting 100 from 11 which becomes -$89, then the IOU manager/owner is switched here and we see the "Pay" button. Until new data comes from the backend and the IOU gets updated properly.

Since this is affecting only the online behavior, where we will always have data coming back from the backend, I'd say we can cancel making updates to the IOU in the client and rely on the backend only.

If we absolutely need this for offline behavior, another solution I can think of is to store currency exchange values in Onyx and use them when we're offline.

@JmillsExpensify what do you think?

[JmillsExpensify]

Gotcha, so I think what we're essentially discovering here is that we can't reliably Cancel in scenarios where an IOU has multiple currencies – just like we recently discovered for Pay. The pickle with the "Pay" is that it's possible to accidentally pay something for an amount you didn't expect. That said, a couple of questions:

• Isn't Cancel only an issue when both currencies were added online, but then one of them is canceled offline? If that's the case, I think it's the same issue as Pay and we should just require you to be online for the Pay button to be enabled. That gives the IOU the chance to update using the correct amounts, and thus total. So you know what you're paying.

• Related, if you Cancel offline, I thought we already agreed that if an IOU has multiple currencies that the total would not be updated offline? So maybe I'm missing why Cancel should also be disabled while offline?

P.S. We looked into storing currencies in the App briefly and there was a huge performance it, so I tend to agree that it's not a realistic solution for now.

[youssef-lr]

Isn't Cancel only an issue when both currencies were added online, but then one of them is canceled offline

The issue here is cancelling a request while online, and when the request's currency is different from the report's. e.g. cancelling 100 AED from a report that shows $27, it will actually subtract 100 from 27 which will flip the IOU status and show a Pay button temporarily. If you notice in the screen recording in this issue, around the 1:32 second mark, the request was cancelled online.

Related, if you Cancel offline, I thought we already agreed that if an IOU has multiple currencies that the total would not be updated offline? So maybe I'm missing why Cancel should also be disabled while offline?

Correct, cancelling offline works fine and there is no need to disable it.

[youssef-lr]

Here's a recording of the bug:

Screen.Recording.2022-11-17.at.18.24.14.mov

[trjExpensify]

I'm kind of confused. The steps in the OP are not the same as the steps in the OP video. Here's the flow following the video:

1. Launch the app
2. Log in with any account
3. Send a money request in USD while being online, e.g. $100. (no prior balance owed between either parties on the report)
4. Go offline
5. Send a second money request in a different currency, e.g. 100 AED. (The report total doesn't update and shows pending) - ✅ correct
6. Go online, the report total updates with the value of the second request (e.g $127.23 is owed by the other user) - ✅ correct
7. Go offline again
8. Cancel the $100 request in USD. The report total updates and shows pending. (e.g $27.23 is owed by the other user) - ✅ correct
9. Go online again, the report total remains as $27.23 owed by the other user - ✅ correct
10. Cancel the 100 AED request online
11. The report total briefly shows you now owe $72.77 with the Pay button - ❌ incorrect
12. Then updates (well, reverts?) to $27.23 owed - ❌ incorrect
13. Then $0 and the report preview is removed as the balance is equal again. - ✅ correct

So the question I have is what is happening to make the report total go from $27.23 to $72.77 when cancelling a request online? Before reverting back to $27.23, before cancelling it to bring the report total to $0.

[youssef-lr]

The report total briefly shows you now owe $72.77 with the Pay button - ❌ incorrect

That's the code in the client (incorrectly) subtracting 100 AED from $27.23 = -$72.77

Then updates (well, reverts?) to $27.23 owed - ❌ incorrect

Not sure what's happening here either. I didn't see this in my testing.

Then $0 and the report preview is removed as the balance is equal again. - ✅ correct

Correct data was fetched from the backend.