Merged in [16688] from sasha@dashcare.nl:
Fix #2478 - When completing review, show which addresses it is posted to.
- Legacy-Id: 16795
Note: SVN reference [16688] has been migrated to Git commit 68ea11a916
This commit is contained in:
commit
8c71252fee
|
@ -610,6 +610,9 @@ class ReviewTests(TestCase):
|
|||
# get
|
||||
r = self.client.get(url)
|
||||
self.assertEqual(r.status_code, 200)
|
||||
self.assertContains(r, assignment.review_request.team.list_email)
|
||||
for author in assignment.review_request.doc.authors():
|
||||
self.assertContains(r, author.formatted_email())
|
||||
|
||||
# faulty post
|
||||
r = self.client.post(url, data={
|
||||
|
|
|
@ -788,6 +788,8 @@ def complete_review(request, name, assignment_id):
|
|||
'form': form,
|
||||
'mail_archive_query_urls': mail_archive_query_urls,
|
||||
'revising_review': revising_review,
|
||||
'review_to': to,
|
||||
'review_cc': cc,
|
||||
})
|
||||
|
||||
def search_mail_archive(request, name, assignment_id):
|
||||
|
|
|
@ -27,6 +27,9 @@
|
|||
system will post the review for you. If you already have posted
|
||||
the review, you can try to let the system find the link to the
|
||||
archive and retrieve the email body.</p>
|
||||
<p>If you enter the review below, the review will be sent
|
||||
to <em>{{ review_to|join:", " }}</em>
|
||||
{% if review_cc %}, with a Cc to <em>{{ review_cc|join:", " }}</em>{% endif %}.</p>
|
||||
{% else %}
|
||||
<p>You can revise this review by entering the results below.</p>
|
||||
{% endif %}
|
||||
|
|
Loading…
Reference in a new issue