Skip to content

(ios) Bug fixes for currency names#765

Merged
robbiehanson merged 6 commits intomasterfrom
ios-currency-names
Oct 30, 2025
Merged

(ios) Bug fixes for currency names#765
robbiehanson merged 6 commits intomasterfrom
ios-currency-names

Conversation

@robbiehanson
Copy link
Contributor

Fixing various bugs related to currencies in iOS:

  1. We recently renamed ANG to XCG, but when looking at the list of currencies in iOS, the new XCG wasn't sorted properly. It was sorted where it was previously. So we now sort the list of currencies by their displayCode.

  2. Currencies with "market names" (e.g. "Argentine Peso (blue market)") weren't being properly translated. So, for example, all the currencies were properly listed in French except these currencies (which were still in English).

  3. French translation was missing "market names". E.g. "blue market". Please double-check my translations (see specific commit for it, and scan for the green code additions)

  4. Currency rename from ANG to XCG is also a rename from "Netherlands Antillean Guilder" to "Caribbean Guilder". We need this change on Android too.

@robbiehanson robbiehanson requested a review from dpad85 October 29, 2025 15:42
Copy link
Member

@dpad85 dpad85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed a few fixes for the French localisation. There are some other translations that need improvement, but I could not find them in the resources file. Are they automatically translated by the OS?

Currency rename from ANG to XCG is also a rename from "Netherlands Antillean Guilder" to "Caribbean Guilder". We need this change on Android too.

The name is handled by Android using the ISO code, and it seems they have not updated yet. I'm not sure it's worth adding specific code just for this case.

@robbiehanson
Copy link
Contributor Author

robbiehanson commented Oct 29, 2025

There are some other translations that need improvement, but I could not find them in the resources file.

There are 2 localization files. One just for the currencies (which generally don't need to be translated, since iOS can handle them), and the other general one (with all the normal strings from the app).

This PR only updates the currency localizations. I was planning on making a separate PR where I update all the translations.

The name is handled by Android using the ISO code, and it seems they have not updated yet.

Same on iOS. They have ANG, but not XCG yet.
There is a way to override the default (iOS provided) currency names on iOS. In this case we're using that override to provide the missing XCG value.

I'm not sure it's worth adding specific code just for this case.

Probably not. If people are mentioning the issue to us, I'm sure they've already mentioned it to Google.

@robbiehanson robbiehanson merged commit 0532893 into master Oct 30, 2025
@robbiehanson robbiehanson deleted the ios-currency-names branch October 30, 2025 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants