-
Notifications
You must be signed in to change notification settings - Fork 448
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
Public masthead page #9860
Comments
@asmecher, could you please review the PRs above? I see there was about > editorialTeam page that is not used any more, correct? What was this, do you remember? The very old masthead from 2.4? EDIT: |
pkp/pkp-lib#9860 Public masthead page
pkp/pkp-lib#9860 Public masthead page
pkp/pkp-lib#9860 Public masthead page
@asmecher, could you please also review these PRs:
|
Thanks, @bozana, please go ahead! In general it's not a good idea to use DAOs in the upgrade scripts, because the DAO may change in the future (e.g. as the schema gets adjusted) but the upgrade scripts should not evolve. But we don't have any plans to change this aspect of the code, so I suppose it's OK. |
#9860 add editorial masthead nav menu item at upgrade
pkp/pkp-lib#9860 add editorial masthead nav menu item at upgrade
pkp/pkp-lib#9860 add editorial masthead nav menu item at upgrade
pkp/pkp-lib#9860 add editorial masthead nav menu item at upgrade
The masthead page should be under context's about.
It should list all active masthead services i.e. all masthead roles as well as uses that are active in those masthead roles and that have accepted to be listed on masthead.
Users are listed with following information: full name, affiliation, start date and orcidIcon (if ORCID authorized).
Reviewer role is treated differently: The reviewers are listed for the previous year. All reviewers that have submitted a review in the previous year will be displayed on the masthead page.
JII plugin should be able to overwrite the masthead users -- generally, a user can opt-out of being displayed on masthead, but this does not guild for JII users -- according to JII all users in the masthead roles needs to be displayed (and cannot opt-out).
PRs:
Upgrade script that adds the Editorial Masthead nav menu item under About nav menu item:
PRs:
The text was updated successfully, but these errors were encountered: