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

Added Serbian Cyrillic translation and corrected Serbian Latin translаtion #2553

Merged
merged 2 commits into from
Oct 24, 2023

Conversation

imilos
Copy link
Contributor

@imilos imilos commented Oct 20, 2023

Description

Added Serbian Cyrillic translation and corrected Serbian Latin translаtion

Instructions for Reviewers

Added sr-cyr.json5 translation and modified config files to include this translation. Also slightly corrected Serbian Latin.

List of changes in this PR:

  • Added Serbian Cyrillic
  • Modified Serbian Latin
  • Updated config.example.yml and default-app-config.ts.

@imilos imilos closed this Oct 20, 2023
@imilos imilos reopened this Oct 20, 2023
@imilos
Copy link
Contributor Author

imilos commented Oct 20, 2023

Corrected linter errors.

@imilos
Copy link
Contributor Author

imilos commented Oct 20, 2023

I am not sure what exactly is failing in 18.x. Anyone help with this?

@tdonohue
Copy link
Member

@imilos : The failure in Node 18.x looks to be a random failure, not caused by this PR. We occasionally have a few tests that fail semi-randomly (which is unfortunate), but we know that they will usually succeed if we rerun the test. I just triggered a re-run of the 18.x tests on this PR.

So, this looks good. I'll pull this over to our 7.6.x maintenance board to find a tester so that we can try to include this in 7.6.1. Thanks!

@tdonohue tdonohue added i18n / l10n Internationalisation and localisation, related to message catalogs 1 APPROVAL pull request only requires a single approval to merge port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release labels Oct 20, 2023
@alanorth alanorth changed the title Added Serbian Cyrrilic translation and corrected Serbian Latin translаtion. Added Serbian Cyrillic translation and corrected Serbian Latin translаtion Oct 23, 2023
@alanorth
Copy link
Contributor

Thanks @imilos. I tested this on DSpace 7.6 and it looks good. It's also good you caught the minor errors in the Serbian Latin strings. Хвала!

@alanorth alanorth self-requested a review October 24, 2023 18:01
@alanorth alanorth merged commit 6affb06 into DSpace:main Oct 24, 2023
17 of 18 checks passed
@dspace-bot
Copy link
Contributor

@alanorth alanorth removed the port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release label Oct 24, 2023
@alanorth alanorth added this to the 8.0 milestone Oct 24, 2023
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
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants