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

Missing locale key site.upgradeAvailable.admin in OMP #9294

Closed
NateWr opened this issue Sep 11, 2023 · 8 comments
Closed

Missing locale key site.upgradeAvailable.admin in OMP #9294

NateWr opened this issue Sep 11, 2023 · 8 comments
Assignees
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone

Comments

@NateWr
Copy link
Contributor

NateWr commented Sep 11, 2023

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() returns true.

  1. Login as admin
  2. Go to Administration area
  3. See error

What application are you using?
OMP main branch running a stable-3_4_0 database.

Additional information
upgrade-available

@asmecher asmecher added this to the 3.4.0-x milestone Jan 3, 2024
@kaitlinnewson kaitlinnewson self-assigned this Jul 29, 2024
@kaitlinnewson
Copy link
Member

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 show_upgrade_warning is not typically set in the OMP config file/template (see also #8543).

kaitlinnewson added a commit to kaitlinnewson/pkp-lib that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 13, 2024
@jonasraoni jonasraoni added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/pkp-lib that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/pkp-lib that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/ojs that referenced this issue Aug 13, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 13, 2024
@asmecher
Copy link
Member

@kaitlinnewson, unfortunately the locale key makes explicit reference to OJS, and would need to be generalized in order to be moved to the pkp-lib shared library.

@kaitlinnewson
Copy link
Member

@kaitlinnewson, unfortunately the locale key makes explicit reference to OJS, and would need to be generalized in order to be moved to the pkp-lib shared library.

🤦‍♀️ Not sure how I missed that! I'll work on generalizing this key.

kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 14, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 26, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 26, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 26, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 26, 2024
kaitlinnewson added a commit to kaitlinnewson/omp that referenced this issue Aug 26, 2024
@kaitlinnewson
Copy link
Member

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

@asmecher
Copy link
Member

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.

@asmecher
Copy link
Member

⬆️ Sorry, I see you specified that these were adapted manually. Should be good to go!

asmecher pushed a commit to pkp/omp that referenced this issue Aug 27, 2024
asmecher pushed a commit to pkp/omp that referenced this issue Aug 27, 2024
asmecher pushed a commit to pkp/omp that referenced this issue Aug 27, 2024
asmecher pushed a commit to pkp/omp that referenced this issue Aug 27, 2024
@asmecher
Copy link
Member

All merged, thanks!

@asmecher asmecher modified the milestones: 3.4.0-x, 3.3.0-20 Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Development

No branches or pull requests

4 participants