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

pkp/pkp-lib#10571 limit email template access by user groups #815

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

taslangraham
Copy link
Contributor

@taslangraham taslangraham changed the title pkp/pkp-lib#10571 limit template access by sender and receiver role pkp/pkp-lib#10571 limit email template access by user groups Nov 20, 2024
@taslangraham taslangraham force-pushed the i10571_limit_template_access_by_sender_and_receiver_role branch 4 times, most recently from 895bd59 to 5c61190 Compare November 20, 2024 17:05
@taslangraham taslangraham marked this pull request as ready for review November 20, 2024 18:31
@taslangraham taslangraham force-pushed the i10571_limit_template_access_by_sender_and_receiver_role branch 2 times, most recently from 1991787 to 7401f85 Compare November 21, 2024 17:53
@@ -135,6 +135,10 @@
<migration class="PKP\migration\upgrade\v3_5_0\COA75_AddUserRoleEndEmail"/>
</upgrade>

<upgrade minversion="3.5.0.0" maxversion="3.5.0.0">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same about the maxversion from OJS. Should be 3.5.9.9.

@taslangraham taslangraham force-pushed the i10571_limit_template_access_by_sender_and_receiver_role branch 2 times, most recently from 03302bb to 7de2a42 Compare November 25, 2024 22:54
@taslangraham taslangraham force-pushed the i10571_limit_template_access_by_sender_and_receiver_role branch from 7de2a42 to 952c0fb Compare November 25, 2024 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants