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.

CommentFileSizeAuthor
#3 874556-mail_optin.patch8.25 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Title: Required opt-in for anonymous notifications » Required opt-in for anonymous notifications with a confirmation mail

Interesting idea. I probably won't work on it, but would review/commit a patch.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Assigned: Unassigned » dawehner

I will work on it.g

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Assigned: dawehner » Unassigned
Status: Active » Needs work
FileSize
874556-mail_optin.patch8.25 KB

Here is a initial version. Sadly it lacks simpletests at the moment.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

It 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.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented

I 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?

LeisureLarry’s picture

LeisureLarry CreditAttribution: LeisureLarry commented

Subscribing