Skip to content

Fix data release translations#10521

Open
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/10275-data-release-translations
Open

Fix data release translations#10521
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/10275-data-release-translations

Conversation

@Montekkundan
Copy link
Copy Markdown
Contributor

Brief summary of changes

  • Translates user-facing Data Release labels and SweetAlert buttons.
  • Adds missing locale entries for file upload, user, and confirmation strings.

Testing instructions (if applicable)

  • Run make data_release.
  • Open Data Release permission and file management forms in supported locales.
  • Confirm File to upload, User, OK, and confirmation buttons are translated.

Link(s) to related issue(s)

@Montekkundan Montekkundan self-assigned this Jun 2, 2026
@github-actions github-actions Bot added 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 Jun 2, 2026
@skarya22 skarya22 assigned skarya22 and unassigned Montekkundan Jun 2, 2026
@skarya22 skarya22 self-requested a review June 2, 2026 17:51
@skarya22 skarya22 added the Passed manual tests PR has been successfully tested by at least one peer label Jun 8, 2026
@skarya22 skarya22 removed their assignment Jun 8, 2026
"Manage Permissions": "権限の管理",
"Note": "注記",
"Version names will be saved as lowercase.": "バージョン名は小文字で保存されます",
"File to upload": "アップロードするファイル",
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.

This file (and all the other json locale ones) shouldn't be committed.. it's generated by make. It should be gitignored.

msgid "Note that the version associated with the file will also be overwritten."
msgstr "ファイルに関連付けられたバージョンも上書きされることに注意してください。"

msgid "Yes, I am sure!"
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.

These both already exist in the loris namespace in the loris.pot

msgid "Upload File"
msgstr "上传文件"

msgid "File to upload"
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.

This also already exists in the loris namespace

<SelectElement
name='userid'
label='User'
label={t('User', {ns: 'data_release'})}
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.

Surprisingly "User" doesn't already exist in the loris namespace, but "Username" does.

Maybe this should be "Username" for consistency across LORIS?

@ridz1208 any thoughts?

@CamilleBeau CamilleBeau added the State: Needs work PR awaiting additional work by the author to proceed label Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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 Passed manual tests PR has been successfully tested by at least one peer State: Needs work PR awaiting additional work by the author to proceed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[data_release] Untranslated phrases in module

4 participants