Send notifications only to users who can view content

Created on 14 September 2021, about 3 years ago
Updated 6 December 2023, 12 months ago

Problem/Motivation

There is an issue Notification emails should not send to blocked users which does make sense because we don't want blocked user to see the content, but what if we are using some modules to restrict user access to content, based on something else than Role or Status? For example Permissions by Term module?

Proposed resolution

Attaching a patch with small fix, which checks 'view' access on entity before add user account to the list of notification recipients.
With this patch we may not need to use the patch from #3078110 issue, since blocked users don't have 'view' access anyway.

Feature request
Status

Fixed

Version

3.0

Component

Code

Created by

🇺🇦Ukraine tarasich Kyiv

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024