681375 - Send out emails for new private messages

Status: VERIFIED FIXED

(support.mozilla.org :: General, defect, P1)

Description

[Kadir Topal [:atopal]]

You should receive an email for every private message, with the text in the email, and there should be a reply link in the private message

Comment 1

[James Socol [:jsocol, :james]]

*Optionally.* There must be a way to turn this off.

Comment 2

[Kadir Topal [:atopal]]

Yes, I think we should have a "settings" section for users, where they can make changes like this.

Comment 3

[[:Cww]]

Minimally, a link at the bottom of the email to stop sending these.

Comment 4

[James Socol [:jsocol, :james]]

s/Minimally/Additionally/. Definitely a requirement. Either it turns them off automatically or links you to the settings page.

Comment 5

[James Socol [:jsocol, :james]]

This depends on having settings, so 9-13 isn't really feasible.

Comment 6

[Kadir Topal [:atopal]]

You can put the preference "send emails for private messages" on the profile page for now. But not sending emails defeats the purpose of private messages. When you want to contact a one-off contributor, the chances for him/her to see the PM are very low without an email notification. Please re-target for some time this quarter.

Comment 7

[Kadir Topal [:atopal]]

Could you please give this top priority now that bug 681935 is fixed? I think this is worth delaying other projects if that is necessary.

Comment 8

[Tim Watts [:timw]]

What copy should it?
For the user preference it's currently: 
Send emails for private messages

The email subject:
You have a new private message from [{sender}]

The email body:
{{ sender }} sent you the following message:{% endblocktrans %}
{{ message|safe }}

https://{{ host }}{{ url }}

Comment 9

[Tim Watts [:timw]]

https://github.com/jsocol/kitsune/pull/355

Comment 10

[Ricky Rosario [:rrosario, :r1cky]]

next train.

Comment 11

[Tim Watts [:timw]]

example:
--------
Subject: You have a new private message from [Tim]
From: [e-mail]
To: [e-mail]
Date: Wed, 30 Nov 2011 00:56:50 -0000
Message-ID: <[e-mail]>

Tim sent you the following message:
Foo bar

https://support-local.allizom.org/en-US/messages/read/17

Comment 12

[Verdi [:verdi] Best to slack me]

The email subject:
You have a new private message from [{sender}]

The email body:
{{ sender }} sent you the following message:{% endblocktrans %}
{{ message|safe }}

View this message on the web:
https://{{ host }}{{ url }}

Change your email preferences:
(link to settings)

Comment 13

[Kadir Topal [:atopal]]

What Michael said, please with one change:

subject: [SUMO] You have a new private message from [{sender}]

Comment 14

[Tim Watts [:timw]]

pushed https://github.com/jsocol/kitsune/commit/02971e57dace411510e4986b7de38e0372cf329e

Comment 15

[Rebecca Billings [:rbillings]]

Verified PM email is sent with correct text, verified links to the message and to the settings page.