-
Notifications
You must be signed in to change notification settings - Fork 447
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
Missing locale key site.upgradeAvailable.admin in OMP #9294
Comments
This looks like a case of just needing to move the locale key to pkp-lib from OJS, but it's not easy to reproduce because |
PRs for review @asmecher: |
@kaitlinnewson, unfortunately the locale key makes explicit reference to OJS, and would need to be generalized in order to be moved to the |
🤦♀️ Not sure how I missed that! I'll work on generalizing this key. |
@asmecher I've updated the list of PRs above, ready for another review. Since we are not actively translating the 3.3 branch, I've copied over the full text from OJS and updated the strings to OMP where possible. For 3.4/main, I made a small change to the english text so we aren't using "here" links to follow better accessibility practices. |
Thanks, @kaitlinnewson, looks good! Were those stable-3_3_0 translations adapted manually from OJS translations, or machine translated, or something else? Generally speaking, it's good to mark them fuzzy so anyone working with translation tools can give them a review. |
⬆️ Sorry, I see you specified that these were adapted manually. Should be good to go! |
All merged, thanks! |
Describe the bug
The locale key,
site.upgradeAvailable.admin
, is not present in OMP. This leads to a missing message in the site administration area.To Reproduce
I'm not sure of the reproduction steps, but I the message is shown when
VersionCheck::checkIfNewVersionExists()
returnstrue
.What application are you using?
OMP
main
branch running astable-3_4_0
database.Additional information
The text was updated successfully, but these errors were encountered: