Skip to content

Commit e83cee2

Browse files
committed
Merged in [12487] from rjsparks@nostrum.com:
Make it clearer what version was requested and what version was reviewed on a reveiwers overview page. Fixes ietf-tools#2101. - Legacy-Id: 12490 Note: SVN reference [12487] has been migrated to Git commit 96390da
1 parent 8c7a400 commit e83cee2

1 file changed

Lines changed: 10 additions & 4 deletions

File tree

ietf/templates/ietfauth/review_overview.html

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ <h2>Assigned reviews</h2>
2121
<table class="table table-condensed table-striped">
2222
<thead>
2323
<tr>
24-
<th>Request</th>
24+
<th>Document</th>
25+
<th>Version Requested</th>
26+
<th>Current Version</th>
2527
<th>Team</th>
2628
<th>Type</th>
2729
<th>Deadline</th>
@@ -30,7 +32,9 @@ <h2>Assigned reviews</h2>
3032
<tbody>
3133
{% for r in open_review_requests %}
3234
<tr>
33-
<td><a {% if r.pk != None %}href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}"{% endif %}>{{ r.doc.name }}-{% if r.requested_rev %}{{ r.requested_rev }}{% else %}{{ r.doc.rev }}{% endif %}</a></td>
35+
<td><a {% if r.pk != None %}href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}"{% endif %}>{{ r.doc.name }}</a></td>
36+
<td>{% if r.requested_rev %}{{ r.requested_rev }}{% else %}Current{% endif %}</td>
37+
<td>{{r.doc.rev}}</td>
3438
<td>{{ r.team.acronym }}</td>
3539
<td>{{ r.type.name }}</td>
3640
<td>
@@ -52,7 +56,8 @@ <h2>Latest closed review requests</h2>
5256
<table class="table table-condensed table-striped">
5357
<thead>
5458
<tr>
55-
<th>Request</th>
59+
<th>Document</th>
60+
<th>Version Reviewed</th>
5661
<th>Team</th>
5762
<th>Type</th>
5863
<th>Deadline</th>
@@ -63,7 +68,8 @@ <h2>Latest closed review requests</h2>
6368
<tbody>
6469
{% for r in closed_review_requests %}
6570
<tr>
66-
<td><a {% if r.pk != None %}href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}"{% endif %}>{{ r.doc.name }}-{% if r.requested_rev %}{{ r.requested_rev }}{% else %}{{ r.doc.rev }}{% endif %}</a></td>
71+
<td><a {% if r.pk != None %}href="{% url "ietf.doc.views_review.review_request" name=r.doc.name request_id=r.pk %}"{% endif %}>{{ r.doc.name }}</a></td>
72+
<td>{{r.reviewed_rev|default:"See review"}}{% if r.requested_rev %}{% if r.requested_rev != r.reviewed_rev %}({{ r.requested_rev }} requested){% endif %}{% endif %}</td>
6773
<td>{{ r.team.acronym }}</td>
6874
<td>{{ r.type.name }}</td>
6975
<td>

0 commit comments

Comments
 (0)