Add Polish desktop client translation - #6974
Conversation
|
|
📝 WalkthroughWalkthroughAdds Polish to the available language list and introduces a complete Polish translation bundle covering client UI, settings, onboarding, updates, runtime states, and authentication messages. ChangesPolish localization
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@client/ui/i18n/locales/pl/common.json`:
- Around line 80-82: Update the message value for the tray.menu.about
translation to a Polish label meaning “About,” such as “O NetBird” or
“Informacje,” rather than “Pomoc i wsparcie.”
- Around line 692-699: Update the Polish messages for
settings.ssh.localForward.help and settings.ssh.remoteForward.help to use the
grammatical “Zezwalaj ... na” construction, replacing the infinitive forms with
“na tunelowanie” and “na udostępnianie” while preserving the existing meaning.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: ad7d7779-4fa5-4640-97fb-76ffb11fa949
📒 Files selected for processing (2)
client/ui/i18n/locales/_index.jsonclient/ui/i18n/locales/pl/common.json
| "tray.menu.about": { | ||
| "message": "Pomoc i wsparcie" | ||
| }, |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Translate the About menu item as About.
tray.menu.about is currently “Pomoc i wsparcie” (“Help and support”), while documentation and troubleshooting already have separate entries. Use a label such as “O NetBird” or “Informacje”.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@client/ui/i18n/locales/pl/common.json` around lines 80 - 82, Update the
message value for the tray.menu.about translation to a Polish label meaning
“About,” such as “O NetBird” or “Informacje,” rather than “Pomoc i wsparcie.”
| "settings.ssh.localForward.help": { | ||
| "message": "Zezwalaj łączącym się urządzeniom tunelować swoje lokalne porty do usług dostępnych z tego hosta." | ||
| }, | ||
| "settings.ssh.remoteForward.label": { | ||
| "message": "Zdalne przekierowanie portów" | ||
| }, | ||
| "settings.ssh.remoteForward.help": { | ||
| "message": "Zezwalaj łączącym się urządzeniom udostępniać na tym hoście porty prowadzące do ich własnych maszyn." |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Use the Polish zezwalać na construction.
Both SSH forwarding descriptions use Zezwalaj ... tunelować/udostępniać, which is grammatically incorrect. Change these to Zezwalaj ... na tunelowanie and ... na udostępnianie.
Proposed wording
- "message": "Zezwalaj łączącym się urządzeniom tunelować swoje lokalne porty do usług dostępnych z tego hosta."
+ "message": "Zezwalaj łączącym się urządzeniom na tunelowanie swoich lokalnych portów do usług dostępnych z tego hosta."
- "message": "Zezwalaj łączącym się urządzeniom udostępniać na tym hoście porty prowadzące do ich własnych maszyn."
+ "message": "Zezwalaj łączącym się urządzeniom na udostępnianie na tym hoście portów prowadzących do ich własnych maszyn."📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "settings.ssh.localForward.help": { | |
| "message": "Zezwalaj łączącym się urządzeniom tunelować swoje lokalne porty do usług dostępnych z tego hosta." | |
| }, | |
| "settings.ssh.remoteForward.label": { | |
| "message": "Zdalne przekierowanie portów" | |
| }, | |
| "settings.ssh.remoteForward.help": { | |
| "message": "Zezwalaj łączącym się urządzeniom udostępniać na tym hoście porty prowadzące do ich własnych maszyn." | |
| "settings.ssh.localForward.help": { | |
| "message": "Zezwalaj łączącym się urządzeniom na tunelowanie swoich lokalnych portów do usług dostępnych z tego hosta." | |
| }, | |
| "settings.ssh.remoteForward.label": { | |
| "message": "Zdalne przekierowanie portów" | |
| }, | |
| "settings.ssh.remoteForward.help": { | |
| "message": "Zezwalaj łączącym się urządzeniom na udostępnianie na tym hoście portów prowadzących do ich własnych maszyn." |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@client/ui/i18n/locales/pl/common.json` around lines 692 - 699, Update the
Polish messages for settings.ssh.localForward.help and
settings.ssh.remoteForward.help to use the grammatical “Zezwalaj ... na”
construction, replacing the infinitive forms with “na tunelowanie” and “na
udostępnianie” while preserving the existing meaning.
2 participants
What changed
pl) locale for the desktop client and tray UIWhy
Polish was not available in the desktop client's language picker despite the existing localization framework.
Terminology
The duration strings use abbreviations where necessary because the current one/other plural model cannot represent all Polish plural forms.
Validation
go test ./client/ui/i18n ./client/ui/preferencesgit diff --checkThe translation has not yet been reviewed in the running desktop UI; this draft PR leaves room for native-language and visual review.
Summary by CodeRabbit