Skip to content

Conversation

@oshaposhnyk
Copy link
Contributor

Description

Short summary of changes (1-2 sentences).

Instructions for Reviewers

The uk lang file contained the name of a specific Ukrainian university, which I removed so that users would not have to search for how to remove it themselves.

@alanorth alanorth added 1 APPROVAL pull request only requires a single approval to merge affects: main Issue impacts "main" (latest release). affects: 8.x Issue impacts 8.x releases affects: 7.x Issue impacts 7.x releases affects: 9.x Issue impacts 9.x releases i18n / l10n Internationalisation and localisation, related to message catalogs port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release and removed affects: main Issue impacts "main" (latest release). affects: 8.x Issue impacts 8.x releases affects: 7.x Issue impacts 7.x releases affects: 9.x Issue impacts 9.x releases labels Sep 2, 2025
@alanorth
Copy link
Contributor

alanorth commented Sep 2, 2025

Thank you @oshaposhnyk! I see this affects DSpace 7, 8, and 9 as well so I will mark this accordingly so the pull request bot can attempt to port it.

Can I also request that you remove the TODO note about changing the university name, since you have addressed that? Thanks!

@tdonohue tdonohue moved this to 🙋 Needs Reviewers Assigned in DSpace 10.0 Release Sep 2, 2025
@oshaposhnyk
Copy link
Contributor Author

@alanorth done

@tdonohue tdonohue added this to the 10.0 milestone Dec 15, 2025
Copy link
Member

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

👍 Thank you @oshaposhnyk ! Apologies it took so long to get back to this PR. But, I had a chance to try it out today and verified that the new translation is showing up properly.

@github-project-automation github-project-automation bot moved this from 🙋 Needs Reviewers Assigned to 👍 Reviewer Approved in DSpace 10.0 Release Dec 15, 2025
@tdonohue tdonohue merged commit 29287ad into DSpace:main Dec 15, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Reviewer Approved to ✅ Done in DSpace 10.0 Release Dec 15, 2025
@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-7_x:

@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-8_x:

@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-9_x:

@tdonohue tdonohue removed port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release labels Dec 15, 2025
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 i18n / l10n Internationalisation and localisation, related to message catalogs

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants