fix(i18n): add missing translations for hi-IN#2254
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
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
📝 WalkthroughWalkthroughThis pull request substantially expands the Hindi locale JSON by replacing many previously-empty placeholder objects with complete translation structures. New and extended keys cover footer navigation, keyboard shortcuts, search (sorting, rate limiting, instant search, Algolia notice, metadata), blog content, settings (data_source, instant search, theme/UX toggles, keyboard shortcuts), errors by HTTP status, common UI verbs, profile/package/readme/provenance/security sections, compare UI (facets, diffs, charts, table view), privacy policy, accessibility, translation status, and new top-level sections like action_bar and vacations. No code or exported/public entities were changed. 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 |
ghostdevv
left a comment
There was a problem hiding this comment.
could you take a look at the code rabbit suggestions? then I can merge!
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
Incorporated suggestions from Coderabbit |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
i18n/locales/hi-IN.json (1)
24-1468:⚠️ Potential issue | 🟡 MinorFix missing
{count}placeholder inabout.contributors.title.The token parity check against
en.jsonfound 1 mismatch: the Hindi translation ofabout.contributors.titleis missing the{count}interpolation token present in the English version. Add the placeholder to the Hindi string to preserve pluralisation support.
🧹 Nitpick comments (1)
i18n/locales/hi-IN.json (1)
235-235: Localise the prefilled invite share text for hi-IN.
profile.invite.compose_textis still fully English, which creates a mixed-language experience in the Hindi locale.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9d9d17c3-ebf7-4171-a13e-2df288951dc4
📒 Files selected for processing (1)
i18n/locales/hi-IN.json
🔗 Linked issue
N/A
🧭 Context
Add missing translations for hi-IN
📚 Description
Add missing translations for hi-IN