add reply tokens in reviewer emails (Bug 879535) #898
Conversation
# Include Mozilla contact. | ||
if thread.read_permission_mozilla_contact: | ||
if thread.addon.mozilla_contact: | ||
recipients.append(thread.addon.mozilla_contact) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be [x.split() for x in thread.addon.mozilla_contact.split(',')]
, right?
looks good; let me know when these things are addressed |
{% block content %} | ||
{% trans -%} | ||
A user posted on a discussion of {{ name }} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when you use a {% trans %}
tag all the whitespace (i.e., newlines) gets collapsed ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was inspired by this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, that example is wrong
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can something like {% blocktrans %}
be used here ?
fix those things and this looks good to merge |
emails, perm_setting='app_reviewed', cc=cc_email, | ||
attachments=self.get_attachments()) | ||
|
||
if self.comm_thread and waffle.switch_is_active('comm_emails'): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waffle name = 'comm-dashboard' :/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eek, good call
r+wc |
@@ -0,0 +1,8 @@ | |||
{% extends 'reviewers/emails/base.txt' -%} | |||
{% block content %} | |||
{{ _('A user posted on a discussion of {0}.', name) }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{{ _('A user posted on a discussion of {name}.')|f(name=name) }}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could probably use an L10n comment.
thread=self.thread, note_type=0, body='something', | ||
read_permission_developer=True) | ||
res = self.client.post(reverse('comm-note-replies-list', | ||
kwargs={'thread_id': self.thread.id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
almost lines up
r+wc |
merged 600e47 |
No description provided.