Add config possibility to disable mailalert

Created on 21 June 2022, over 2 years ago
Updated 3 June 2023, over 1 year ago

Problem/Motivation

When migrating from Drupal 7 to Drupal 9, it will begin to do comment notifications, which isn't suitable for our usecase. Therefore it would be great to be able to disable the mailalert through config.

Steps to reproduce

Install a site with comment_notify and setup a migration for comments and comment notify. When migrating comments comment_notify will begin to sent out mail notifications.

Proposed resolution

Add a disable possibility to config and use it in mailalert.

Remaining tasks

Approve attached patch.

User interface changes

Added the config as part of the settings form.

API changes

None.

Data model changes

Expanded the comment_notify.settings.yml

Feature request
Status

Needs work

Version

1.0

Component

Code

Created by

🇩🇰Denmark pmchristensen

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.

  • This should not only be a global configuration option, but rather one that can be changed temporarily and programmatically by other modules that want to copy or convert content nodes and comments.

    For example, if some module wants to convert a node to a different type, all comments need to be copied. This now triggers a comment notification for all users involved in a discussion that may be years old.

    If there is a quick hack to prevent this, I'd like to know. Using some global Drupal variable should do the trick, I suppose.

Production build 0.71.5 2024