Change the default template for assigned review emails to name the assigned reviewer. Fixes ietf-tools#2830. Commit ready for merge.

 - Legacy-Id: 17062

Author: rjsparks

ietf/dbtemplate/migrations/0007_adjust_review_assigned.py

@@ -0,0 +1,38 @@
+# Copyright The IETF Trust 2019, All Rights Reserved
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.26 on 2019-11-19 11:47
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+def forward(apps, schema_editor):
+    DBTemplate = apps.get_model('dbtemplate','DBTemplate')
+    qs = DBTemplate.objects.filter(path='/group/defaults/email/review_assigned.txt')
+    qs.update(content="""{{ assigner.ascii }} has assigned {{ reviewer.person.ascii }} as a reviewer for this document.
+
+{% if prev_team_reviews %}This team has completed other reviews of this document:{% endif %}{% for assignment in prev_team_reviews %}
+- {{ assignment.completed_on }} {{ assignment.reviewer.person.ascii }} -{% if assignment.reviewed_rev %}{{ assignment.reviewed_rev }}{% else %}{{ assignment.review_request.requested_rev }}{% endif %} {{ assignment.result.name }}  
+{% endfor %}
+""")
+    qs.update(title="Default template for review assignment email")
+
+def reverse(apps, schema_editor):
+    DBTemplate = apps.get_model('dbtemplate','DBTemplate')
+    qs = DBTemplate.objects.filter(path='/group/defaults/email/review_assigned.txt')
+    qs.update(content="""{{ assigner.ascii }} has assigned you as a reviewer for this document.
+
+{% if prev_team_reviews %}This team has completed other reviews of this document:{% endif %}{% for assignment in prev_team_reviews %}
+- {{ assignment.completed_on }} {{ assignment.reviewer.person.ascii }} -{% if assignment.reviewed_rev %}{{ assignment.reviewed_rev }}{% else %}{{ assignment.review_request.requested_rev }}{% endif %} {{ assignment.result.name }}  
+{% endfor %}
+""")
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('dbtemplate', '0006_add_review_assigned_template'),
+    ]
+
+    operations = [
+        migrations.RunPython(forward, reverse)
+    ]

ietf/doc/tests_review.py

@@ -430,7 +430,7 @@ def test_assign_reviewer(self):
         self.assertEqual(len(outbox), 1)
         self.assertEqual('"Some Reviewer" <reviewer@example.com>', outbox[0]["To"])
         message = outbox[0].get_payload(decode=True).decode("utf-8")
-        self.assertIn("{} has assigned you".format(secretary.person.ascii), message)
+        self.assertIn("{} has assigned {}".format(secretary.person.ascii, reviewer.person.ascii), message)
         self.assertIn("This team has completed other reviews", message)
         self.assertIn("{} -01 Serious Issues".format(reviewer_email.person.ascii), message)