Skip to content

[data_release] Add missing translations + adapted code#10772

Open
HachemJ wants to merge 7 commits into
aces:29.0-releasefrom
HachemJ:AddMissingTranslationsDataRelease
Open

[data_release] Add missing translations + adapted code#10772
HachemJ wants to merge 7 commits into
aces:29.0-releasefrom
HachemJ:AddMissingTranslationsDataRelease

Conversation

@HachemJ

@HachemJ HachemJ commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Brief summary of changes

This PR adds missing translation of the data_release module and fixes a french translation requested in the issue.
For the 'Unversioned' related code changes, I would prefer if someone could confirm that I updated the correct query for the user-facing translation and that I didn’t miss any related code paths that should be changed or introduce any regressions.

Note: I used DeepL (AI) to translate the strings in hindi, japanese and chinese.

Link(s) to related issue(s)

@github-actions github-actions Bot added Language: PHP PR or issue that update PHP code Language: Javascript PR or issue that update Javascript code Module: data_release PR or issue related to data_release module Multilingual Any tasks related to making LORIS multilingual labels Jul 6, 2026
@HachemJ HachemJ added the 29.0.0 - Bugs Issues found during the release testing for 29.0.0 label Jul 6, 2026
msgstr "ファイルの管理"

msgid "Unversioned"
msgstr "バージョン管理対象外"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Google Translate says this means "Not Under Version Control" which seems a little suspicious.

It translates "Unversioned" as "バージョンなし" (which translates back to English as "Version None") which might be more accurate for a file since this isn't about using a version control system?

I'm not fluent enough in Japanese to say which sounds more natural but intuitively the Google Translate version seems to match the usage here more closely.

// the downloaddir.
return new \LORIS\Http\Response\JSON\InternalServerError(
$e->getMessage()
dgettext("data_release", $e->getMessage())

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same concern as described in my other PR #10863 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

29.0.0 - Bugs Issues found during the release testing for 29.0.0 Language: Javascript PR or issue that update Javascript code Language: PHP PR or issue that update PHP code Module: data_release PR or issue related to data_release module Multilingual Any tasks related to making LORIS multilingual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants