Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Port dspace-7_x] use translation service to determine placeholder values of date picker #3965

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

tdonohue
Copy link
Member

@tdonohue tdonohue commented Feb 6, 2025

Manual port of #2827 by @saschaszott to dspace-7_x

DSpace#2827)

* use translation service to determine placeholder values

* added translation of date-picker placeholder values

* added German translations of date-picker placeholder values

* fix lint warning: added missing comma

* fix lint warning: removed trailing spaces

* fixed broken tests

* fixed lint warning

* try to mock TranslateService correctly

* remove TranslateService mock

* removed imports

* fixed lint warnings

* fixed lint warnings

* fixed lint warning

* fixed lint warning

* fixed lint error

* fixed lint errors

* removed TestComponent

* added imports

* removed declarations

* fixed lint error

* fix lint error (invalid sorting)

* finally fixing lint error

* use translation pipe directly in template

* move translation of placeholders into HTML template

* remove TranslateService dependency
@tdonohue tdonohue added bug i18n / l10n Internationalisation and localisation, related to message catalogs component: submission 1 APPROVAL pull request only requires a single approval to merge labels Feb 6, 2025
@tdonohue tdonohue added this to the 7.6.3 milestone Feb 6, 2025
Copy link
Member Author

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

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

👍 Because I had to make minor changes to get this to work on 7.x, I've tested it manually and found it works

@tdonohue tdonohue merged commit 7fab963 into DSpace:dspace-7_x Feb 6, 2025
15 checks passed
@tdonohue tdonohue deleted the port_2827_to_7x branch February 6, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge bug component: submission i18n / l10n Internationalisation and localisation, related to message catalogs
Projects
Development

Successfully merging this pull request may close these issues.

2 participants