Enable Russian localization - #1036
Mangalushka wants to merge 1 commit into
Conversation
The ru/ locale files have shipped since webadderallorg#446, but "ru" was never added to SUPPORTED_LOCALES, so the bundle was never imported and Russian never showed up in the language picker. Register the locale, wire its namespaces into I18nContext, add the label, and translate the 22 strings added to settings after the original contribution (click effects and the caption editor rows). npm run i18n:check passes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: webadderallorg/Recordly/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe application now includes Russian in its supported locales and displays “Русский” in the language selector. Russian bundles are registered for seven namespaces, and selected click-effect and caption-editor strings are translated. ChangesRussian locale support
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to Russian can be selected and its registered translations are available for the app’s namespaces. The changed settings strings match their corresponding keys, and no concrete merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description explains the purpose, affected files, translation changes, validation command, and implementation notes. It does not follow the repository template and omits explicit Type of Change, Related Issue(s), Screenshots / Video, and Checklist sections.
✨ 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 |
What
Russian translations have been in the repo since #446 (May), but
ruwas never added toSUPPORTED_LOCALES, so the bundle is never imported and the language never appears in the picker. This wires it up:src/i18n/config.ts— registerrusrc/contexts/I18nContext.tsx— import the sevenrunamespaces and add them tomessagessrc/components/video-editor/SettingsPanel.tsx— addРусскийtoAPP_LANGUAGE_LABELSsrc/i18n/locales/ru/settings.json— translate the 22 strings added after the original contribution (the click-effects block and the caption editor rows), which were still EnglishChecks
npm run i18n:check→ locale files are structurally consistentpt-BRoutsidelocales/), so this is the whole wiringNotes
This rebases #691 onto current
main(that PR conflicts now, anddehas been added since). Credit to @Domanffe for the original enablement and @voinkovpost for the translations. Happy to close this if #691 gets rebased instead.🤖 Generated with Claude Code
Summary by CodeRabbit