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

Add possibility to not send emails for users that have emailstop=1 #173

Open
wants to merge 1 commit into
base: MOODLE_405_STABLE
Choose a base branch
from

Conversation

mattiabonzi
Copy link

This update introduces a new feature to the mod_reengagement plugin, allowing the exclusion of users who have the "emailstop" preference set to true from receiving emails.

  • The functionality is controlled by a flag at the activity level.
  • By default, the flag is set to false to preserve the existing behaviour, ensuring emails are sent to all users as before.

Add possibility to not send emails for users that have emailstop=1
@danmarsden
Copy link
Member

Hi @mattiabonzi this feels a little strange to be implemented within the plugin itself - the plugin uses moodle's core api "message_send" which already checks emalstop.

can you provide a bit more information about how users are getting emails when they have emailstop enabled? - the core api should be preventing it.

Looking through the code I do wonder if the "thirdpartyemails" setting is set and it's those users who are getting emails despite the fact they have emailstop enabled? - in which case the patch you've added won't help either.

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