Repository navigation
i18n(fr): add machine translations for all missing keys - #1565
Open
bhuvan-somisetty wants to merge 2 commits into
Open
bhuvan-somisetty wants to merge 2 commits into
bhuvan-somisetty wants to merge 2 commits into
Conversation
bhuvan-somisetty
added a commit
to bhuvan-somisetty/jam
that referenced
this pull request
Oct 22, 2026
Contributor
Author
|
this one sits on top of #1541, so only the last commit is new here. I'll rebase once that's merged. added before/after screenshots too |
bhuvan-somisetty
force-pushed
the
i18n/fr-fill-missing
branch
from
October 22, 2026 22:28
13c56d9 to
4f11ab1
Compare
Contributor
Author
|
small fix: my merge script had split the |
fr only had 157 of the 874 keys, so most of the app fell back to English. Fills in the 717 missing keys (plus the `many` plural forms fr needs) with machine translations. Existing translations are left as they are. The file now follows the key order of en. Signed-off-by: bhuvan-somisetty <somisettybhuvan5@gmail.com>
Signed-off-by: bhuvan-somisetty <somisettybhuvan5@gmail.com>
bhuvan-somisetty
force-pushed
the
i18n/fr-fill-missing
branch
from
November 4, 2026 10:53
f3b42cb to
1d9a450
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fills in all missing French translations with machine translations, as discussed with @theborakompanioni (AI translation beats falling back to English; anyone who finds a better wording can improve it with a PR).
manyplural forms fr needs for large numbersen, which makes future diffs easier to compareBuilds on #1541: that PR prunes stale fr keys and adds the locale checks (variables,
<Trans>components, plural forms) this translation was validated against. Only the twoi18n(fr): ...commits are new here (the second one keepsdescription_tx_fees_factor_^0.9.10a flat key like in en); I'll rebase once #1541 is merged.Testing
src/i18n/locales.test.tspasses for fr: every{{ variable }},<n>component and$t()reference matchesen, and all fr plural forms (one,many,other) are present0 → Aucun UTXO dans le pot A,1 → 1 UTXO dans le pot A,2 → 2 UTXOs dans le pot A, and nested hints like "Un balayage planifié nécessite que vos UTXOs aient 5 confirmations ou plus. Attendez 2 blocs de plus."prettier --checkis cleanVisual Demo (For contributors especially)
Image Demo (if applicable):
Storybook with locale
fr(before =devel, after = this PR):