i18n(fr): update UI strings#119
Conversation
WalkthroughThis update adds new French translation entries to two localization JSON files. The Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Tip ⚡💬 Agentic Chat (Pro Plan, General Availability)
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/content/i18n/fr.json (1)
47-48: Ensure key naming & punctuation consistency
- The key
"quick-update"diverges from the dot‑notation pattern used elsewhere (e.g."docsearch.noResultsScreen.reportMissingResultsLinkText"). Confirm it aligns with your i18n lookup conventions or consider renaming (e.g."content.quickUpdate").- The instruction string ends without a colon. Since it introduces a command, you might add one for clarity:
"Mettez rapidement à jour vers la dernière version en exécutant la commande suivante :"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/content/i18n/fr.json(1 hunks)src/starlight-sidebar/fr.json(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Run Starlight Link Check
| "auto-gen": "Génération auto", | ||
| "topic-guides": "Guides et tutoriels", | ||
| "database": "Guides des bases de données", | ||
| "upgrade": "Mise à niveau", | ||
| "version": "Guides des versions", | ||
| "latest": "Guide de la dernière mise à jour" |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Verify integration of new sidebar translation keys
You’ve added keys ("topic-guides", "database", "upgrade", "version", "latest"). Please ensure the sidebar component actually references these, for example by running:
to catch any missing usages.
🏁 Script executed:
rg -nE '"topic-guides"|"database"|"upgrade"|"version"|"latest"' -C2 srcLength of output: 165
🏁 Script executed:
#!/bin/bash
# Re-run search with correct ripgrep flags to locate uses of the new translation keys
rg -n -e '"topic-guides"' -e '"database"' -e '"upgrade"' -e '"version"' -e '"latest"' -C2 srcLength of output: 2602
Add references to the new sidebar translation keys in the component
The keys ("topic-guides", "database", "upgrade", "version", "latest") are defined in each src/starlight-sidebar/*.json but aren’t referenced anywhere in the codebase. You’ll need to update your sidebar component (e.g. wherever you build the menu items) to use these keys via your i18n lookup.
• Translation files:
- src/starlight-sidebar/en.json (lines 16–20)
- src/starlight-sidebar/fr.json (lines 16–20)
- src/starlight-sidebar/es.json (lines 16–20)
- src/starlight-sidebar/de.json (lines 16–20)
- src/starlight-sidebar/ko.json (lines 16–20)
Please integrate these keys into the sidebar configuration or remove them if they’re not needed.
dreyfus92
left a comment
There was a problem hiding this comment.
Thank you @ArmandPhilippot 🫡
Description
Updates French UI strings (
fr.jsonandstarlight-sidebar/fr.json) with changes from #73.Summary by CodeRabbit