Problem/Motivation

#3232943: Send notifications only to users who can view content implemented a change where emails are not sent unless the user being sent to has access to view the entity.

I think this should be conveyed in the form prominently. It's otherwise frustrating to debug why emails are not being sent.

I also think this change should not affect the adhoc emails. Sometimes a listserv email/group email is used for adhoc emails. Emails to those addresses are delivered to many individuals that DO have access to view the content, but the email is never sent. The only solution is to register an account on the site with that email address and assign it privileges to view the content, but that's not really desireable in all situations.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork content_moderation_notifications-3409111

Command icon 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:

About issue forks

Comments

bkosborne created an issue. See original summary.

tyapchyc’s picture

tyapchyc commented
Assigned: Unassigned » tyapchyc

Tyapchyc opened merge request !21

tyapchyc’s picture

tyapchyc commented
Status: Active » Needs review

Added information about access to view the entity to fields on notification form.

tyapchyc’s picture

tyapchyc commented
Assigned: tyapchyc » Unassigned
tyapchyc’s picture

tyapchyc commented

Hello @rob-holmes, @jhedstrom

Could you please review this?

Thank you

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Status: Needs review » Reviewed & tested by the community

This looks good to me. This just bit me again 2 years later. This description would have helped. Thanks.

  • tyapchyc committed 8d55b3ab on 8.x-3.x 
    Issue #3409111: Update notification form to indicate users being emailed...
tyapchyc’s picture

tyapchyc commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.