Merged in [12596] from rjsparks@nostrum.com:
Show other completed reviews of a document on page showing the status of a review request. Fixes #2083.
- Legacy-Id: 12605
Note: SVN reference [12596] has been migrated to Git commit 2bad0b0bcd
This commit is contained in:
commit
24c9751df4
|
@ -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'])
|
||||
|
||||
|
|
|
@ -67,6 +67,18 @@
|
|||
</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>
|
||||
|
|
Loading…
Reference in a new issue