Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It would be cool if there would be an option,
which would allow to first mail to the user a notification-enable link.
Sure this shouldn't be the default setting.
Comment | File | Size | Author |
---|---|---|---|
#3 | 874556-mail_optin.patch | 8.25 KB | dawehner |
Comments
Comment #1
gregglesInteresting idea. I probably won't work on it, but would review/commit a patch.
Comment #2
dawehnerI will work on it.g
Comment #3
dawehnerHere is a initial version. Sadly it lacks simpletests at the moment.
Comment #4
dawehnerIt shouldn't be called initial version. The only thing that is missing is from my perspective the simpletest. I tested the patch some hours already.
Comment #5
gregglesI think the language could be expanded.
You wanted to get notifications for !node_title. If you go to !link1 you subscribe to new comments.
How about "A comment at !comment-link included a request to subscribe to comments. If you requested that, go to !link1.
If you didn't request this, you can ignore this e-mail."
Also, I'm not sure I like the idea of using the same notify hash to enable comments. Maybe we should restrict it so that it will only do the update for comments that have been posted in the last 48 hours? Otherwise someone could use this feature to re-enable notifications months after the fact.
I'd also really like to get this into 7.x first, could you do that?
Comment #6
LeisureLarry CreditAttribution: LeisureLarry commentedSubscribing