Skip to content

Conversation

@adi-herwana-nus
Copy link
Contributor

No description provided.

@adi-herwana-nus adi-herwana-nus force-pushed the adi/consolidate-backend-translations branch from 5755f74 to a439fa4 Compare December 26, 2025 00:24
@adi-herwana-nus adi-herwana-nus force-pushed the adi/consolidate-backend-translations branch 3 times, most recently from 24ba2cc to c0795e5 Compare January 9, 2026 01:13
@adi-herwana-nus adi-herwana-nus force-pushed the adi/consolidate-backend-translations branch from c0795e5 to 00aec35 Compare January 9, 2026 01:17
Consolidate scattered error translations into dedicated errors.yml files
for EN, KO, and ZH locales. This improves maintainability by organizing
all error messages in a single location per language.

Changes:
- Created config/locales/{en,ko,zh}/errors.yml with all error translations
- Updated 16 backend files to use new translation key paths with errors. prefix
- Deleted 33 component-specific translation files (11 per language)
- Fixed activerecord error structure in KO and ZH to match EN hierarchy

All translation keys now follow the pattern: errors.{domain}.{component}.{key}

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@adi-herwana-nus adi-herwana-nus force-pushed the adi/consolidate-backend-translations branch from 00aec35 to 2d0a466 Compare January 9, 2026 01:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR consolidates backend translation files for Chinese (zh), Korean (ko), and English (en) locales by merging scattered YAML files into centralized structures. The changes improve maintainability by organizing translations into fewer, more cohesive files.

Changes:

  • Consolidated activerecord error messages and attributes into unified errors.yml and attributes.yml files
  • Merged mailer and notifier translations into a central mailers.yml file
  • Created csv.yml for CSV export headers and values
  • Updated application code references to use the new translation key paths

Reviewed changes

Copilot reviewed 300 out of 343 changed files in this pull request and generated no comments.

Show a summary per file
File Description
config/locales/{zh,ko,en}/activerecord/errors.yml Consolidates all activerecord error messages from multiple separate files
config/locales/{zh,ko,en}/activerecord/attributes.yml Consolidates all activerecord attribute translations from multiple files
config/locales/{zh,ko,en}/mailers.yml Merges course mailers, instance mailers, and notifiers into one file
config/locales/{zh,ko,en}/csv.yml Centralizes all CSV export-related translations
config/locales/{zh,ko,en}/errors.yml Creates unified error messages for authentication, course, and user modules
app/views/**/*.{slim,erb} Updates translation key references to new consolidated paths
app/controllers/**/*.rb Updates translation key references in controllers
app/services/**/*.rb Updates translation key references in services
app/models/**/*.rb Updates translation key references in models
app/helpers/**/*.rb Updates translation key references in helpers

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants