Skip to content

Add i18n locales#460

Open
cathysarisky wants to merge 4 commits intoTryGhost:mainfrom
cathysarisky:add-i18n-locales
Open

Add i18n locales#460
cathysarisky wants to merge 4 commits intoTryGhost:mainfrom
cathysarisky:add-i18n-locales

Conversation

@cathysarisky
Copy link
Member

This PR adds a new locales folder to the shared theme assets. New build tools (see PR in Source repo) will retrieve these locales for assembling a locales file for each theme.

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

A new English locale JSON file has been added at packages/_shared/assets/locales/en.json. The file contains key-value pairs for UI strings covering site navigation, post interactions, subscription prompts, and access-related messages. All translation values are currently empty strings awaiting translation content. This is a structural addition for internationalization support with no functional logic changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add i18n locales' directly matches the main change—adding a new locales folder with English locale strings.
Description check ✅ Passed The description accurately explains the purpose of adding locales and mentions the new build tools that will use them.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cathysarisky cathysarisky requested a review from 9larsons February 5, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant