IOU - Currencies are not displayed after clicking the currency symbol for a second time

[isagoico]

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

---

Expected Result:

List of currencies should display.

Actual Result:

An empty list is displayed.

Action Performed:

1. Open e.cash and log in
2. Navigate to a conversation
3. Click on request money
4. Click on the currency symbol
5. Select a currency
6. Click on the currency symbol.

Workaround:

Searching for the currency does show the results.

Platform:

Where is this issue occurring?

Web ✔️
iOS ✔️
Android
Desktop App
Mobile Web

Version Number: 1.0.66-0

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

Notes/Photos/Videos:

Expensify/Expensify Issue URL:

View all open jobs on Upwork

---

From @AndrewGable https://expensify.slack.com/archives/C01GTK53T8Q/p1623251127130300

On the second time you change the currency on an IOU the currency list is blank. e.g. click $ -> change to £ -> click £ -> List is empty

[MelvinBot]

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

[OSBotify]

👋 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.

[trjExpensify]

@yuwenmemon, I confirmed this isn't happening on production, so pretty sure it must be related to one of the PRs from yesterday's staging deploy here.

[kidroca]

This will be resolved after Expensify/react-native-onyx#81 is merged and react-native-onyx updated

[yuwenmemon]

Looks like it got merged!

[kidroca]

We still need to update E.cash, will post a PR in a second

[yuwenmemon]

Thanks @kidroca!

[Julesssss]

Oh, I've got a bump PR here too @kidroca

[Julesssss]

Nevermind, you've seen already 👍

[kidroca]

@yuwenmemon
The merge request with a fix for this issue was merged by @Julesssss

[Julesssss]

Removing the DeployBlocker label

[kidroca]

@isagoico @Julesssss
I'm not sure why this ticket is still open
I'm not working on anything related to Currencies

[isagoico]

Not sure either. I just rechecked the issue in Web and it's not happening anymore. Closing the issue (@Julesssss please feel free to reopen if this issue was open for other reason)

[Julesssss]

Whoops, this was missed somehow. Thanks for updating it!