Invoices - Bank account row is missing currency after reset and restart app

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: 9.0.89-2
Reproducible in staging?: Y
Reproducible in production?: No, new feature
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team
Component: Workspace Settings

Action Performed:

1. Go to staging.new.expensify.com
2. Go to workspace settings > Invoices
3. Click Add bank account
4. Select any non-US country (eg, India)
5. Go through the setup to add an India bank account
6. Note that it shows INR before the dot separator in the bank account row
7. Go to account settings > Troubleshoot > Clear cache and restart
8. Click Reset and restart
9. Do not go to Wallet
10. Go to workspace settings > Invoices
11. Note that the bank account row is missing currency before the dot separator
12. Go to account settings > Wallet
13. Go back to workspace settings > Invoices
14. Now it shows INR before the dot separator in the bank account row

Expected Result:

In Step 11, the bank account row should show the currency after reset and restart the app

Actual Result:

In Step 11, the bank account row is missing currency after reset and restart the app.
The currency only appears after opening Wallet page

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Standalone
• Android: HybridApp
• Android: mWeb Chrome
• iOS: Standalone
• iOS: HybridApp
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6722176_1737666816318.20250124_050434.mp4

View all open jobs on GitHub

[melvin-bot]

Triggered auto assignment to @tgolen (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[melvin-bot]

💬 A slack conversation has been started in #expensify-open-source

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[lanitochka17]

A new feature (different behavior on prod)

bandicam.2025-01-24.00-28-40-328.mp4

[mkzie2]

OpenApp API is missing bankCurrency in bankAccountList:

This also happens when you sign out and sign in again.

However, we can apply an improvement in the FE to only show the currency (and the dot separator) when it exists as mentioned in my proposal here #55675 (comment).

[Beamanator]

@MonilBhavsar @shubham1206agra @DylanDylann is this NAB b/c of the same beta as #55672 (comment)?

[shubham1206agra]

@Beamanator This is a BE bug. Can you mark this internal?

[Beamanator]

Okkk @shubham1206agra can you explain a bit more why you say so? I may be able to help fix if i know the prob :D

[shubham1206agra]

@Beamanator See #55674 (comment) for explanation.

[Beamanator]

Shmart!

[Beamanator]

Looks like @MonilBhavsar worked that in the backend recently, maybe you want to take this over

[MonilBhavsar]

Yep, feature is under beta and looks backend. I'll take over

[MonilBhavsar]

Sent a PR

[melvin-bot]

@MonilBhavsar Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[MonilBhavsar]

PR was deployed. Closing this one