Skip to content

Split i18n YAML files #4791

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Split i18n YAML files #4791

wants to merge 23 commits into from

Conversation

@veganstraightedge
Copy link
Contributor

@veganstraightedge veganstraightedge commented Aug 17, 2025 •
edited
Loading

  • Split up en.yml into separate smaller files in en/ folder
  • Re-organized the location of page titles in locales
  • Expanded locale yaml for all of the models
  • Updated #admin_title method to match the new locale structure
  • Moved admin/en.yml into en/admin.yml
  • Moved admin/es.yml into es/admin.yml
  • Moved formats/es.yml into es/formats.yml

@veganstraightedge veganstraightedge marked this pull request as ready for review August 20, 2025 18:56
veganstraightedge
veganstraightedge commented Aug 20, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@astronaut-wannabe astronaut-wannabe Awaiting requested review from astronaut-wannabe astronaut-wannabe is a code owner

@bookis bookis Awaiting requested review from bookis bookis is a code owner

@eliduke eliduke Awaiting requested review from eliduke eliduke is a code owner

@evanphx evanphx Awaiting requested review from evanphx evanphx is a code owner

@guitsaru guitsaru Awaiting requested review from guitsaru guitsaru is a code owner

@just1602 just1602 Awaiting requested review from just1602 just1602 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@veganstraightedge