528994 - [gallery] Put the jetpack name in the review notification subject

Status: VERIFIED FIXED

(Websites Graveyard :: jetpackgallery.mozillalabs.com, defect, P3)

Description

[Jeff Balogh (:jbalogh)]

The current subject is: "Your Jetpack has been reviewed..."

While that builds up a lot of suspense to see what's inside, it would be nice to have the jetpack name in the subject.  Adding the review body would be extra special.

Comment 1

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Added the subject line.

I also would personally like to see the review body in the email.  But I think we want to encourage developers to visit the site in order to view and interact with the comments.  If the developer reads it in their email, they are much less likely to visit the site, and thus much less likely to interact with the users who reviewed their jetpack, IMHO.

==

Sending        application/helpers/notify.php
Transmitting file data .
Committed revision r57567.

Comment 2

[Jeff Balogh (:jbalogh)]

(In reply to comment #1)
> Added the subject line.
> 
> I also would personally like to see the review body in the email.  But I think
> we want to encourage developers to visit the site in order to view and interact
> with the comments.  If the developer reads it in their email, they are much
> less likely to visit the site, and thus much less likely to interact with the
> users who reviewed their jetpack, IMHO.

I'd like that too.  Trying to force people onto the site sounds like a facebook trick.  As long as there's an obvious link to the comment, I don't think I'd be less likely to interact with users.

Comment 3

[krupa raj[:krupa]]

verified