Merged in [12596] from rjsparks@nostrum.com:

    Show other completed reviews of a document on page showing the status of a review request. Fixes ietf-tools#2083.
 - Legacy-Id: 12605
Note: SVN reference [12596] has been migrated to Git commit 2bad0b0

Author: levkowetz

ietf/review/models.py

@@ -131,6 +131,12 @@ class ReviewRequest(models.Model):
     def __unicode__(self):
         return u"%s review on %s by %s %s" % (self.type, self.doc, self.team, self.state)
 
+    def other_requests(self):
+        return self.doc.reviewrequest_set.exclude(id=self.id)
+
+    def other_completed_requests(self):
+        return self.other_requests().filter(state_id__in=['completed','part-completed'])
+
 def get_default_review_types():
     return ReviewTypeName.objects.filter(slug__in=['early','lc','telechat'])
 

ietf/templates/doc/review/review_request.html

@@ -67,6 +67,18 @@ <h1>Review request<br><small>{{ review_req.doc.name }}</small></h1>
         </tr>
       {% endif %}
 
+      <tr>
+        <th></th>
+        <th>Other Reviews</th>
+        <td>
+          {% for req in review_req.other_completed_requests %}
+            {% if req.reviewer == review_req.reviewer %}<strong>{% endif %}
+            <a href="{% url "ietf.doc.views_review.review_request" name=req.doc request_id=req.id %}">{{req.team.acronym|capfirst}} {{req.type.name}} review of -{{req.reviewed_rev}} by {{req.reviewer.person.plain_name}}</a> {% if req.reviewed_rev != req.doc.rev %} (<a href="{{ rfcdiff_base_url }}?url1={{ req.doc.name }}-{{ req.reviewed_rev }}&url2={{ req.doc.name }}-{{ req.doc.rev }}">diff</a>){% endif %}</br>
+            {% if req.reviewer == review_req.reviewer %}</strong>{% endif %}
+          {% endfor %}
+        </td>
+      </tr>
+
       {% if review_req.comment %}
         <tr>
           <th></th>