Problem/Motivation
Emails to addresses listed under adhoc emails are only sent if the module can verify a user account with that email has access to view the entity OR if anonymous users have access to view the content.
I find this behavior unexpected. For adhoc emails, I don't think there should be an expectation that the module does this user account lookup and access check process.
This feature was implemented in #3232943: Send notifications only to users who can view content#9 and a comment from there expressed the same. I also was previously very confused by this and created #3409111: Update notification form to indicate users being emailed must have access to view the content to at least improve the inline documentation to help avoid pitfalls.
Steps to reproduce
Proposed resolution
Let's either remove the validation for adhoc emails entirely (preferable), OR make validation configurable.
Remaining tasks
User interface changes
API changes
Data model changes
Issue fork content_moderation_notifications-3548983
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
bkosborneMR submitted that just removes the access check for adhoc emails. I'll fix the tests too, but before I spend time on that I'd like to get maintainer buy-in on this path. Thank you.
Comment #5
bkosborneSee also #3419565: Allow a notification to bypass the content access check where this was requested. Adding contrib credit to that patch author.
Comment #6
bkosborne