Skip to content

feat: Turkish localization support#19

Open
ibayazit wants to merge 3 commits intoholistics:mainfrom
ibayazit:feat/turkish-language-support
Open

feat: Turkish localization support#19
ibayazit wants to merge 3 commits intoholistics:mainfrom
ibayazit:feat/turkish-language-support

Conversation

@ibayazit
Copy link
Copy Markdown

Summary

  • Added full localization support for the Turkish (tr) language.
  • Duplicated the English locale structure, updated language metadata, and provided translations for all YAML files.
  • Registered the new locale in the main entry point and verified the production build.

Issue

Lasting Changes (Technical)

  • Locale Addition: Created the locales/tr/ directory with translated strings derived from the English source.
  • Metadata Update: Added Turkish language entries to locales/metadata.yaml.
  • Language Registration: Updated src/index.js to import the Turkish locale, enabling it within the application.
  • Build Validation: Ran yarn build to ensure new locale imports and YAML files are correctly bundled without errors.

Checklist

Please check directly on the box once each of these are done

  • Documentation (if necessary)
  • Lint Checks Passed
  • Unit Tests Passed
  • Coverage Tests Passed
  • Integration Tests Passed
  • Code Review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces Turkish (tr) localization by adding a comprehensive set of translation files covering account management, AI features, authentication, billing, and general UI components. The src/index.js file was also updated to register the new locale. The review feedback identifies several areas for improvement, including addressing empty translation keys, translating remaining English strings, and refining the idiomatic flow and grammar of specific Turkish phrases to ensure a high-quality user experience.

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