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

Send approval admin notification emails to all users with the pmpro_approver role or pmpro_approvals capability. #179

Open
ipokkel opened this issue Sep 21, 2023 · 2 comments

Comments

@ipokkel
Copy link
Member

ipokkel commented Sep 21, 2023

Is your feature request related to a problem? Please describe.
Only the default site administrator receives the admin_approved, admin_denied, and admin_notification_approval approval notification emails. In some cases, the responsibility to approve memberships is delegated to various user accounts by either assigning them the Membership Approver - pmpro_approver role or the Membership Manager - pmpro_membership_manager role that included the pmpro_approvals capability. These users do not receive these aforementioned emails.

Describe the solution you'd like
All approval admin notification emails are sent to all users who have the pmpro_approver role or a role that has the pmpro_approvals capability.

Describe alternatives you've considered
Sending a BCC copy to users with custom code - https://gist.github.com/andrewlimaza/1fefe6bcbd929c4bdb0648345a15ccc8

Additional context
Moderators: #519363

@ipokkel
Copy link
Member Author

ipokkel commented Sep 21, 2023

Here's an example code recipe as proof of concept that should achieve this - https://gist.github.com/ipokkel/45992b04e1bccba2cfe412678241f347

@andrewlimaza
Copy link
Collaborator

While this is a good idea, I think we need to cross this type of bridge within core first and let site owners choose what email address to send PMPro emails to.

Thanks for sharing the gist, I am going to leave this issue open for reference and definitely recommend using that gist if customers ask for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants