Skip to content

[raisinbread] Add test translations#10466

Merged
driusan merged 1 commit intoaces:mainfrom
driusan:RaisinbreadTestLocales
Apr 27, 2026
Merged

[raisinbread] Add test translations#10466
driusan merged 1 commit intoaces:mainfrom
driusan:RaisinbreadTestLocales

Conversation

@driusan
Copy link
Copy Markdown
Collaborator

@driusan driusan commented Apr 24, 2026

This adds the pot files and Japanese po files I used for testing translations on the candidate_list page.

pot files were generated by the script in #10448, while test translation strings mostly come from Google Translate.

To use them, you need to manually copy the raisinbread locale to the project/locale directory and run msgfmt -o table.mo table.po for each.

This is not currently done by the raisinbread refresh script which only deals with the database.

@github-actions github-actions Bot added RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset Multilingual Any tasks related to making LORIS multilingual labels Apr 24, 2026
@regisoc regisoc self-assigned this Apr 24, 2026
Copy link
Copy Markdown
Collaborator

@regisoc regisoc left a comment

Choose a reason for hiding this comment

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

@driusan not sure what I am missing, what text should be changed?

This is what I am seeing from my VM:
Image

And here is the content of the project/locale/ folder.

image

Also, some deps needed PHP8.4. Not sure if that is expected or not.

This adds the pot files and Japanese po files I used for testing
translations on the candidate_list page.

pot files were generated by the script in aces#10448, while test translation
strings mostly come from Google Translate.

To use them, you need to manually copy the raisinbread locale to the
project/locale directory and run `msgfmt -o table.mo table.po` for each.

This is not currently done by the raisinbread refresh script which only
deals with the database.
@driusan driusan force-pushed the RaisinbreadTestLocales branch from 1beea9f to ec3ca3e Compare April 27, 2026 14:57
@driusan
Copy link
Copy Markdown
Collaborator Author

driusan commented Apr 27, 2026

@regisoc I just rebased so that some of the code changes that make reference to it which are merged should be included in this branch now.

The ones that are currently merged are: cohort, entity_type, visit filters, and sex in the filters and the data.

@driusan
Copy link
Copy Markdown
Collaborator Author

driusan commented Apr 27, 2026

From your screenshot it looks like you might need to run make dev as well because 32213db was the last commit on the branch before my rebase but it doesn't seem to be included in the screenshot.

Copy link
Copy Markdown
Collaborator

@regisoc regisoc left a comment

Choose a reason for hiding this comment

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

Looks good now

Image

This is a test translation, it should have other PRs after this one for a full process, right?

@driusan driusan merged commit da5d53e into aces:main Apr 27, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Multilingual Any tasks related to making LORIS multilingual RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants