fix(i18n): update polish translations#2029
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
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
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates the Polish locale JSON (i18n/locales/pl-PL.json) with many new and expanded translation keys. It adds footer.blog and footer.builders_chat; extends shortcuts.instant_search and settings.instant_search; introduces a detailed blog subtree including atproto comment/action texts; replaces an empty error object with structured codes (401, 404, 500, 503, default); expands common view_on mappings and UI actions; enlarges profile (including invite); adds pds/npmx.social; and updates downloads, header.compare, file_changes, trends and other UI strings. Total change set: +119/−14 lines. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
♻️ Duplicate comments (2)
i18n/locales/pl-PL.json (2)
107-110:⚠️ Potential issue | 🟡 MinorExpand these counters to the full Polish plural pattern.
These four strings still only provide 3 forms, while this locale uses 5-form Polish pluralisation elsewhere. That can produce the wrong inflection for some counts when the runtime resolves
many/othercategories.
1220-1220:⚠️ Potential issue | 🟡 MinorCapitalise “Atmosphere” in this string.
The product name is capitalised elsewhere in this locale file, so the lower-case form reads as a typo in the UI.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 49a143c9-0762-4e21-a4c0-13f317a40bbf
📒 Files selected for processing (1)
i18n/locales/pl-PL.json
There was a problem hiding this comment.
♻️ Duplicate comments (1)
i18n/locales/pl-PL.json (1)
107-107:⚠️ Potential issue | 🟡 Minor
blog.atproto.reply_countstill needs the fifth Polish plural variant.Line 107 has only four variants, while the Polish locale in this file consistently uses five. That leaves one category collapsing onto the wrong form at runtime. Please mirror the same 5-slot pattern already used by
like_count,repost_count, andmore_replies.💡 Proposed fix
- "reply_count": "{count} odpowiedzi | {count} odpowiedź | {count} odpowiedzi | {count} odpowiedzi", + "reply_count": "{count} odpowiedzi | {count} odpowiedź | {count} odpowiedzi | {count} odpowiedzi | {count} odpowiedzi",
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 8147a42d-d9ae-44f6-9a25-f9d323f4af4d
📒 Files selected for processing (1)
i18n/locales/pl-PL.json
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
📚 Description
Updating polish translations