There may be times when you don't want the module to send mails but you do want it to respond to requests to disable mails. There should be a setting on the configuration page to disable the sending of mails from this module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kscheirer’s picture

Status: Active » Postponed (maintainer needs more info)

Patches welcome :)

kscheirer’s picture

Status: Postponed (maintainer needs more info) » Active
MorinLuc0’s picture

Added the feature to disable the email sending part of the notify comments.
Possible the next step would be some sort of block or page to display the list of notifications the user has, possibly expose the data to views ?

greggles’s picture

Status: Active » Needs review

The patch looks good to me. Could you try adding a test for the behavior to ensure it keeps working?