Send different notifications to watchers and OP when answer is posted

VERIFIED FIXED in 2.6.1

Status

support.mozilla.org
Questions
P1
normal
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Cww, Assigned: erik)

Tracking

unspecified
2.6.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

7 years ago
Right now we send the same text to both watchers of a question and to the asker.  As part of making the emails more personal, we should send different ones.
(Reporter)

Comment 1

7 years ago
Created attachment 508624 [details]
new answer sent to OP
(Reporter)

Comment 2

7 years ago
Created attachment 508625 [details]
new answer sent to watchers

And here are the email texts.  (If we don't have a username, as in the anon case, we should delete the Hi {{username}} line)
(Reporter)

Comment 3

7 years ago
Created attachment 508626 [details]
email when an answer is marked as solution

And another email for when an answer is marked as a solution
(Assignee)

Comment 4

7 years ago
This shouldn't be a problem: just do a switch in _mails() according to the Question instance stored on the Event.
Attachment #508624 - Attachment mime type: application/octet-stream → text/plain
If I read comment 4 correctly this is a fairly small change but not a trivial text change. => 2.6.1
OS: Linux → All
Hardware: x86 → All
Target Milestone: --- → 2.6.1
Assignee: nobody → erik
Priority: -- → P1
(Assignee)

Comment 6

7 years ago
Cww, are you sure you want to take the question title out of the subject?
(Assignee)

Comment 7

7 years ago
Landed in http://github.com/jsocol/kitsune/commit/d33a794 and http://github.com/jsocol/kitsune/commit/7ae1cbbabef.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Testing blocked by bug #639422
Verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.