feat(i18n): Add Turkish translation#903
Conversation
✅ Deploy Preview for livecodes ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughAdded Turkish ("tr") to the AppLanguage union and appLanguages mapping, and introduced comprehensive Turkish localization files for UI strings and language/tool metadata (translations and Lokalise JSON data). No runtime logic changes. Changes
Sequence Diagram(s)(omitted — changes are data additions and a type/registration update; no control-flow changes to illustrate) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 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 |
|
That's great. Would you please run |
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/livecodes/i18n/locales/tr/language-info.ts (1)
5-14: Consider comment language consistency across locale files.The ATTENTION comment and template comments in this file are in Turkish, while the corresponding comments in
translation.tsare in English.If these are auto-generated files, verify whether the comment language should be consistent across all Turkish locale files or if this variation is intentional.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
src/livecodes/i18n/locales/tr/language-info.lokalise.json(1 hunks)src/livecodes/i18n/locales/tr/language-info.ts(1 hunks)src/livecodes/i18n/locales/tr/translation.lokalise.json(1 hunks)src/livecodes/i18n/locales/tr/translation.ts(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- src/livecodes/i18n/locales/tr/language-info.lokalise.json
🚧 Files skipped from review as they are similar to previous changes (1)
- src/livecodes/i18n/locales/tr/translation.lokalise.json
🧰 Additional context used
🧬 Code graph analysis (2)
src/livecodes/i18n/locales/tr/translation.ts (1)
src/livecodes/i18n/locales/models.ts (1)
I18nTranslationTemplate(23-25)
src/livecodes/i18n/locales/tr/language-info.ts (2)
src/livecodes/html/index.ts (1)
languageInfo(125-125)src/livecodes/i18n/locales/models.ts (1)
I18nTranslationTemplate(23-25)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (11)
- GitHub Check: Codacy Static Code Analysis
- GitHub Check: tests (18.x, 2)
- GitHub Check: tests (18.x, 5)
- GitHub Check: tests (18.x, 4)
- GitHub Check: tests (18.x, 1)
- GitHub Check: tests (18.x, 3)
- GitHub Check: build (18.x)
- GitHub Check: build
- GitHub Check: Redirect rules - livecodes
- GitHub Check: Header rules - livecodes
- GitHub Check: Pages changed - livecodes
🔇 Additional comments (3)
src/livecodes/i18n/locales/tr/translation.ts (2)
1-14: LGTM! File structure and imports are correct.The file header, type imports, and template comments follow the expected pattern for locale files in this codebase.
16-1136: Address formatting issues as requested by the reviewer.The translation structure and type assertion are correct. However, as noted in the PR comments, please run
npm run fixto resolve the formatting errors identified in the CI run.Based on PR comments.
src/livecodes/i18n/locales/tr/language-info.ts (1)
16-447: LGTM! Structure is correct.The language info object structure and type assertion are properly implemented. As with
translation.ts, please ensurenpm run fixis run to address any formatting issues.Based on PR comments.
|
Thank you @ahmed-atiah Please keep up the great contributions. |
i18n ActionsSource PR has been merged into the default branch. Maintainers can comment |
|
.i18n-update-push |
i18n Actions:
|
| Name | Description |
|---|---|
| New Branch for i18n | i18n/ahmed-atiah/develop |
| Last Commit SHA | 5759dd2 |
Maintainers can comment .i18n-update-pull after translation is done to trigger the i18n pull workflow and pull the changes back to Github.
|
.i18n-update-pull |
i18n Actions:
|
| Name | Description |
|---|---|
| i18n Branch | i18n/ahmed-atiah/develop |
| Last Commit SHA | 10328d6 |
| i18n PR | #907 |



What type of PR is this? (check all applicable)
Description
This PR adds support for the Turkish (tr) language to LiveCodes. It includes the creation of all necessary locale files inside i18n/locales/tr/ and registers the language in the application's configuration.
Related Tickets & Documents
I18n #902
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentations?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?
Summary by CodeRabbit