Skip to content

[dictionary] Fix "Visit" and "Cohort" filters in non-English#10995

Open
driusan wants to merge 2 commits into
aces:29.0-releasefrom
driusan:DictionaryFix2
Open

[dictionary] Fix "Visit" and "Cohort" filters in non-English#10995
driusan wants to merge 2 commits into
aces:29.0-releasefrom
driusan:DictionaryFix2

Conversation

@driusan

@driusan driusan commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

This fixes the "Visit" and "Cohort" filters in the non-English user languages.

The filters were being translated but the data was not, so this translates the table values.

Also do not sort "Visit" in the filter so that they show up in the correct order in Japanese.

This fixes the "Visit" and "Cohort" filters in the non-English user languages.

The filters were being translated but the data was not, so this translates the
table values.

Also do not sort "Visit" in the filter so that they show up in the correct
order in Japanese.
@driusan driusan added Module: dictionary PR or issue related to (new) dictionary module Multilingual Any tasks related to making LORIS multilingual 29.0.0 - Bugs Issues found during the release testing for 29.0.0 labels Jul 23, 2026
@github-actions github-actions Bot added Language: PHP PR or issue that update PHP code Language: Javascript PR or issue that update Javascript code labels Jul 23, 2026

@shonibare shonibare 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

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: dictionary PR or issue related to (new) dictionary module Multilingual Any tasks related to making LORIS multilingual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants