Skip to content

MENDELU/Added messages for a new search filter#1262

Merged
milanmajchrak merged 1 commit intocustomer/mendelufrom
mendelu/add-search-filter-accessioned-messages
Mar 23, 2026
Merged

MENDELU/Added messages for a new search filter#1262
milanmajchrak merged 1 commit intocustomer/mendelufrom
mendelu/add-search-filter-accessioned-messages

Conversation

@milanmajchrak
Copy link
Collaborator

Problem description

Analysis

(Write here, if there is needed describe some specific problem. Erase it, when it is not needed.)

Problems

(Write here, if some unexpected problems occur during solving issues. Erase it, when it is not needed.)

Sync verification

If en.json5 or cs.json5 translation files were updated:

  • Run yarn run sync-i18n -t src/assets/i18n/cs.json5 -i to synchronize messages, and changes are included in this PR.

Manual Testing (if applicable)

Copilot review

  • Requested review from Copilot

Copilot AI review requested due to automatic review settings March 23, 2026 13:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds i18n message keys to support a new dateAccessioned search filter in the UI (labels, placeholders, and applied-filter range text) in both English and Czech translation bundles.

Changes:

  • Add dateAccessioned filter UI strings (head/placeholder/label + min/max range labels) to en.json5.
  • Add corresponding Czech translations (with English comment lines) to cs.json5.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/assets/i18n/en.json5 Adds English strings for the new dateAccessioned search filter (UI + applied range labels).
src/assets/i18n/cs.json5 Adds Czech translations for the same keys, following the file’s “commented English source” pattern.

@milanmajchrak milanmajchrak merged commit fb11b40 into customer/mendelu Mar 23, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants