Skip to content

[examiner] Missing certification status options translations - #11153

Open
HachemJ wants to merge 3 commits into
aces:29.0-releasefrom
HachemJ:MissedTranslationsInExaminer
Open

HachemJ wants to merge 3 commits into
aces:29.0-releasefrom
HachemJ:MissedTranslationsInExaminer

Conversation

@HachemJ

@HachemJ HachemJ commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Brief summary of changes

This PR translates the certification status options in the Edit Examiner form to French and updates the change log to display user-facing labels instead of backend enum values.

Testing instructions (if applicable)

Prerequisite: In your config.xml, set EnableCertification to 1 and configure at least one valid certification instrument.

e.g.

<EnableCertification>1</EnableCertification>
...
<CertificationInstruments>
                <test value="bmi">BMI</test>
                <test value="radiology_review">Radiology Review</test>
</CertificationInstruments>
  1. Go to 'Clinical' -> 'Examiner'
  2. Click on an examiner’s name to open the Edit Examiner page
  3. Switch the language to French
  4. Verify that the certification status dropdowns are displayed in French
  5. Verify that the Old Value and New Value columns in the Change Log (table) display the translated certification statuses
  6. Confirm that the Old Value and New Value columns display user-facing labels (e.g., 'Not Certified' in English or 'Non certifié' in French) instead of the backend enum values (e.g., not_certified)
image

Link(s) to related issue(s)

@github-actions github-actions Bot added Language: PHP PR or issue that update PHP code Module: examiner PR or issue related to examiner module Multilingual Any tasks related to making LORIS multilingual labels Aug 5, 2026
@HachemJ HachemJ added the 29.0.0 - Bugs Issues found during the release testing for 29.0.0 label Aug 5, 2026
@HachemJ
HachemJ changed the base branch from main to 29.0-release August 5, 2026 19:12

@lapadulamichael lapadulamichael left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@lapadulamichael lapadulamichael added the Passed manual tests PR has been successfully tested by at least one peer label Sep 16, 2026
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: PHP PR or issue that update PHP code Module: examiner PR or issue related to examiner module Multilingual Any tasks related to making LORIS multilingual Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[examiner] Missed translation in certification logic for date picker and certification status

2 participants