Text of comment +1
Text of post mmm... +0.5
Unsubscribe link: essential
The rest of the junk: non-essential

I'm assuming a lot of this can be configured but I'm talking about the default (which I'm also assuming Dries used on his site).

Colors
Uploaded with plasq's Skitch!
CommentFileSizeAuthor
#4 322165-comment-notify-mail-text-D7.patch1.85 KBDave Reid
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
Status: Active » Postponed (maintainer needs more info)

It's configurable by the admin.

If you want to change the default, please propose text.

Robert - I really value your feedback on improving the UI of this module, but please install it and use it before posting more.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Version: 6.x-1.1 » 7.x-1.x-dev
Status: Closed (works as designed) » Active

I'd like to re-open this for the D7 branch as I think probably most users would benefit from revising the default text.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
322165-comment-notify-mail-text-D7.patch1.85 KB

Here's what I think would be a useful default text. Uses comment title and body for both node and comment notification e-mails. Also uses a very similar signature to the user account e-mails in user.module.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Category: bug » task
Status: Needs review » Fixed

Looks great to me. Committed to 7.x. Thanks, Dave! http://drupal.org/cvs?commit=350840

If anyone has further ideas for improvement, please let's keep them coming.

greggles’s picture

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

Whoops - that was the wrong commit - http://drupal.org/cvs?commit=350846

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.