Skip to content

feat(i18n): update and align French translation - #1939

Closed
comeback01 wants to merge 1 commit into
QuantumNous:mainfrom
comeback01:traduction
Closed

feat(i18n): update and align French translation#1939
comeback01 wants to merge 1 commit into
QuantumNous:mainfrom
comeback01:traduction

Conversation

@comeback01

@comeback01 comeback01 commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Bug Fixes
    • Improved French translations for greater clarity and accuracy.
    • Corrected punctuation in the channel note (dot inside quotes).
    • Fixed leading space issue in a status message.
    • Clarified wording for webhook secret usage (“jeton Bearer”) in headers.
    • Reformatted displayed JSON example in the deployment region input message for better readability.

@coderabbitai

coderabbitai Bot commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Updated French localization values in web/src/i18n/locales/fr.json: revised phrasing for a May 10, 2025 channel note, adjusted spacing on one string, reformatted an embedded JSON example’s line breaks, and clarified “Bearer” wording to “jeton Bearer.” No keys added or removed.

Changes

Cohort / File(s) Summary
French locale updates
web/src/i18n/locales/fr.json
Updated several translation values: clarified dot-in-quotes note for May 10, 2025; added leading space to one string; reflowed an embedded JSON snippet; replaced “Bearer” with “jeton Bearer” in webhook secret descriptions. No keys changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • Calcium-Ion

Poem

A nibble, a nudge, a tiny tweak—
I hop through strings the whole long week.
Quotes aligned, a token named,
JSON neat and whitespace tamed.
With twitching ears I mark it clear:
Bonjour, locale—your updates are here! 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly conveys that this pull request updates and aligns the French translations in the i18n files, directly reflecting the core change described in the summary without extraneous details or vague phrasing. It follows a clear conventional commit format and gives a teammate immediate context on the primary change.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0b91e45 and c0bfa9b.

📒 Files selected for processing (1)
  • web/src/i18n/locales/fr.json (4 hunks)
🔇 Additional comments (4)
web/src/i18n/locales/fr.json (4)

519-519: LGTM! Translation update is clear and accurate.

The French translation correctly conveys that channels added after May 10, 2025 no longer need to remove the dot from model names during deployment. The phrasing is natural and technically precise.


697-697: Verify intentional leading space in translation.

The translation now includes a leading space before "Disponible". While this might be intentional for UI formatting, leading spaces in translation values can sometimes cause inconsistent display or unexpected layout issues.

Please confirm:

  1. Is the leading space intentional for this specific UI context?
  2. Does the UI component properly handle this spacing?
  3. Is this consistent with other similar translations?

You can verify by checking how this string is rendered in the UI and whether the space serves a specific formatting purpose.


1769-1772: LGTM! JSON example formatting is correct.

The reformatted JSON snippet is properly structured with newlines (\n) for readability. The French instructions and example configuration for deployment regions are clear and technically accurate.


1966-1966: LGTM! Improved clarification of Bearer token terminology.

Changing "Bearer" to "jeton Bearer" (Bearer token) enhances clarity for French-speaking users by explicitly identifying "Bearer" as a token type. This is particularly helpful for non-technical users and maintains consistency with French technical documentation standards.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Biome (2.1.2)
web/src/i18n/locales/fr.json

The --json option is unstable/experimental and its output might change between patches/minor releases.
{"summary":{"changed":0,"unchanged":1,"matches":0,"duration":{"secs":0,"nanos":27050181},"scannerDuration":{"secs":0,"nanos":3168814},"errors":1506,"warnings":0,"skipped":0,"suggestedFixesSkipped":0,"diagnosticsNotPrinted":0},"diagnostics":[{"category":"parse","severity":"error","description":"unexpected character .","message":[{"elements":[],"content":"unexpected character ."}],"advices":{"advices":[]},"verboseAdvices":{"advices":[]},"location":{"path":{"file":"web/src/i18n/locales/fr.json"},"span":[38664,38665],"sourceCode":"{\n "主页": "Accueil",\n "控制台": "Console",\n "$%.6f 额度": "Quota de $%.6f",\n "或": "ou",\n "登 录": "Se connecter",\n "注 册": "S'inscrire",\n "使用 邮箱或用户名 登录": "Connectez-vous avec votre e-mail ou votre nom d'utilisateur",\n "使用 GitHub 继续": "Continuer avec GitHub",\n "使用 OIDC 继续": "Continuer avec OIDC",\n "使用 微信 继续": \

... [truncated 99999319 characters] ...

uillez contacter l'administrateur pour configurer Uptime dans les paramètres système.",\n "确定要删除此API信息吗?": "Êtes-vous sûr de vouloir supprimer ces informations d'API ?",\n "Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue.

When opening the issue, please provide a minimal reproduction, or identify and share the file/code that triggers it. Without a way to reproduce the error, the error can't be fixed:

Source Location: crates/biome_console/src/lib.rs:151:14
Thread Name: main
Message: called Result::unwrap() on an Err value: Os { code: 32, kind: BrokenPipe, message: "Broken pipe" }


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

  • Public repositories are always opted into early access features.
  • You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

Comment @coderabbitai help to get the list of available commands and usage tips.

@comeback01 comeback01 closed this Sep 30, 2025
@comeback01
comeback01 deleted the traduction branch September 30, 2025 14:32
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.

1 participant