Skip to content

[schedule_module] Fix filters not working because of table entries not being translated - #11059

Open
HachemJ wants to merge 1 commit into
aces:29.0-releasefrom
HachemJ:FixMultilingualBugRelatedtoDataNotTranslatedScheduleModule
Open

[schedule_module] Fix filters not working because of table entries not being translated#11059
HachemJ wants to merge 1 commit into
aces:29.0-releasefrom
HachemJ:FixMultilingualBugRelatedtoDataNotTranslatedScheduleModule

Conversation

@HachemJ

@HachemJ HachemJ commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Brief summary of changes

This PR fixes a multilingual bug where filters wouldn't work properly due to table entries not being translated (but filters options were).

Testing instructions (if applicable)

  1. git checkout HachemJ/FixMultilingualBugRelatedtoDataNotTranslatedScheduleModule
  2. delete project/locale
  3. copy raisinbread/locale into project/locale
  4. run make dev (make sure it ran the msgfmt --use-fuzzy -o table.mo table.po commands)
  5. Go to 'Tools' -> 'Schedule Appointment'
  6. Switch the language to French
  7. Try the filters and verify that they are working properly (e.g. in French, choose Site: Montréal)

Link(s) to related issue(s)

@github-actions github-actions Bot added Language: PHP PR or issue that update PHP code Module: schedule_module PR or issue related to the schedule_manager module labels Jul 27, 2026
@HachemJ HachemJ added Multilingual Any tasks related to making LORIS multilingual 29.0.0 - Bugs Issues found during the release testing for 29.0.0 labels Jul 27, 2026
@racostas
racostas self-requested a review July 27, 2026 20:15
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: schedule_module PR or issue related to the schedule_manager module Multilingual Any tasks related to making LORIS multilingual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants