-
Notifications
You must be signed in to change notification settings - Fork 445
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
Upgrade script for existing default masthead roles and user assignments #10041
Comments
…assignments to be displayed on masthead
@asmecher, could you please review the PRs above? |
Please go ahead, @bozana. My only thought is that we haven't considered OMP, where there are also internal reviewers that should possibly be listed. It might be worth checking with John about that. |
…assignments to be displayed on masthead
Good catch, @asmecher! -- I currently do not differentiate between internal and external reviewers for display -- I just consider all reviewers from |
#10041 Consider existing default masthead roles and all us…
pkp/pkp-lib#10041 Upgrade script for existing default masthead roles and user assignments
pkp/pkp-lib#10041 Upgrade script for existing default masthead roles and user assignments
pkp/pkp-lib#10041 Upgrade script for existing default masthead roles and user assignments
Existing default masthead roles (editor, section editor and reviewer) should be considered to be displayed on the masthead.
All existing user-role assignments -- not only assignments to the masthead roles, but to all roles -- should be also considered to be displayed on the masthead.
PRs:
The text was updated successfully, but these errors were encountered: