Skip to content

feat: add french translations#480

Draft
TimoGlastra wants to merge 1 commit intomainfrom
feat/french-translations
Draft

feat: add french translations#480
TimoGlastra wants to merge 1 commit intomainfrom
feat/french-translations

Conversation

@TimoGlastra
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Timo Glastra <timo@animo.id>
@TimoGlastra TimoGlastra changed the title chore: feat: add french translations Jan 28, 2026
@TimoGlastra
Copy link
Copy Markdown
Member Author

@claude add new french translations to the app. Make sure to read the intrustions for adding a new language (in readme of translation package). Then extract the language for french and translation all the messages. Then based on the transaltions set up an AI_INSTRUCTIONS specific to french based on what makes sense in how french people would say things. Compare the other AI_INSTRUCTIONS to see what should be added. Then based on the AI instructions, do another pass for constitency.

@claude
Copy link
Copy Markdown

claude bot commented Jan 28, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@TimoGlastra
Copy link
Copy Markdown
Member Author

@claude add new french translations to the app. Make sure to read the intrustions for adding a new language (in readme of translation package). Then extract the language for french and translation all the messages. Then based on the transaltions set up an AI_INSTRUCTIONS specific to french based on what makes sense in how french people would say things. Compare the other AI_INSTRUCTIONS to see what should be added. Then based on the AI instructions, do another pass for constitency.

@claude
Copy link
Copy Markdown

claude bot commented Jan 28, 2026

Claude finished @TimoGlastra's task in 26m 7s —— View job


Adding French Translations - Complete ✓

I've successfully added comprehensive French translations to the app! However, I encountered a permissions issue when trying to push the changes to the remote repository.

Tasks Completed

  • Read translation package instructions
  • Check existing AI_INSTRUCTIONS files for reference
  • Add French to lingui.config.js
  • Add French to index.ts
  • Add French to commonMessages.ts
  • Add French to i18n.ts
  • Create French locale directory
  • Extract French translations (471 messages)
  • Translate all 471 messages to French
  • Merge translations into messages.json
  • Create French-specific AI_INSTRUCTIONS.MD
  • Review consistency based on AI instructions
  • Compile translations and format code
  • Commit changes locally

What Was Done

1. Translation Setup

  • Added French (fr) to all required configuration files
  • Created complete French locale directory structure

2. Comprehensive Translations

  • Translated all 471 messages with proper French terminology
  • Used informal "tu/toi/ton" form for friendly, accessible tone
  • Applied French punctuation rules (space before ?, !, :, ;)
  • Key terminology used:
    • portefeuille (wallet)
    • carte (card/credential)
    • code PIN
    • biométrie (biometrics)
    • organisation
    • partager (share)

3. AI Instructions Document

  • Created apps/easypid/src/locales/fr/AI_INSTRUCTIONS.MD
  • Comprehensive guidelines for future French translations
  • Based on patterns from Dutch, German, Portuguese, and Finnish instructions

4. Consistency Review

  • Fixed 16 terminology inconsistencies
  • Ensured "identifiant" → "carte" throughout
  • Verified grammar and gender agreement
  • Confirmed natural French phrasing

5. Build & Format

  • Successfully compiled translations (0 missing)
  • Formatted code with biome (16 files fixed)

Commit Ready

All changes are committed locally:

commit fe59407
Author: claude[bot]
Date: Wed Jan 28 16:31:05 2026 +0000

feat: add complete French translations to the app

- Added French (fr) locale to lingui.config.js, i18n.ts, commonMessages.ts, and index.ts
- Translated all 471 messages to French with proper terminology and style
- Created French-specific AI_INSTRUCTIONS.MD with translation guidelines
- Reviewed and ensured consistency across all translations
- Used informal "tu/toi/ton" form throughout for friendly, accessible tone
- Key terminology: portefeuille (wallet), carte (card), code PIN, biométrie
- Applied French punctuation rules (space before ?, !, :, ;)
- Compiled and formatted all translation files

Co-authored-by: Timo Glastra <TimoGlastra@users.noreply.github.com>

Files Changed:

  • 21 files changed, 4174 insertions(+), 282 deletions(-)
  • Created: apps/easypid/src/locales/fr/AI_INSTRUCTIONS.MD
  • Created: apps/easypid/src/locales/fr/messages.json
  • Created: apps/easypid/src/locales/fr/messages.ts

Permission Issue

I was unable to push the changes to the remote repository due to GitHub Actions permissions:

remote: Permission to animo/paradym-wallet.git denied to github-actions[bot].

Next Steps:
The commit is ready and can be pushed manually by someone with appropriate repository permissions, or the GitHub Actions permissions can be updated to allow pushes. All the work is complete and committed locally on the branch.

--- |

@Wicpar
Copy link
Copy Markdown
Contributor

Wicpar commented Feb 1, 2026

@claude You don't seem to have pushed any translations

@Wicpar
Copy link
Copy Markdown
Contributor

Wicpar commented Apr 19, 2026

this drifted a lot since the last time, i have pushed #522

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