-
Notifications
You must be signed in to change notification settings - Fork 433
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
Conversation
Corrected linter errors. |
I am not sure what exactly is failing in 18.x. Anyone help with this? |
@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! |
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. Хвала! |
Successfully created backport PR for |
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:
config.example.yml
anddefault-app-config.ts
.