Skip to content

Commit 8c71252

Browse files
committed
Merged in [16688] from sasha@dashcare.nl:
Fix ietf-tools#2478 - When completing review, show which addresses it is posted to. - Legacy-Id: 16795 Note: SVN reference [16688] has been migrated to Git commit 68ea11a
2 parents 25ac3b6 + 68ea11a commit 8c71252

3 files changed

Lines changed: 8 additions & 0 deletions

File tree

ietf/doc/tests_review.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -610,6 +610,9 @@ def test_complete_review_upload_content(self):
610610
# get
611611
r = self.client.get(url)
612612
self.assertEqual(r.status_code, 200)
613+
self.assertContains(r, assignment.review_request.team.list_email)
614+
for author in assignment.review_request.doc.authors():
615+
self.assertContains(r, author.formatted_email())
613616

614617
# faulty post
615618
r = self.client.post(url, data={

ietf/doc/views_review.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -788,6 +788,8 @@ def complete_review(request, name, assignment_id):
788788
'form': form,
789789
'mail_archive_query_urls': mail_archive_query_urls,
790790
'revising_review': revising_review,
791+
'review_to': to,
792+
'review_cc': cc,
791793
})
792794

793795
def search_mail_archive(request, name, assignment_id):

ietf/templates/doc/review/complete_review.html

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,9 @@ <h1>{% if revising_review %}Revise{% else %}Complete{% endif %} review<br>
2727
system will post the review for you. If you already have posted
2828
the review, you can try to let the system find the link to the
2929
archive and retrieve the email body.</p>
30+
<p>If you enter the review below, the review will be sent
31+
to <em>{{ review_to|join:", " }}</em>
32+
{% if review_cc %}, with a Cc to <em>{{ review_cc|join:", " }}</em>{% endif %}.</p>
3033
{% else %}
3134
<p>You can revise this review by entering the results below.</p>
3235
{% endif %}

0 commit comments

Comments
 (0)