diff --git a/ietf/group/tests_review.py b/ietf/group/tests_review.py index 0e56c9f46f..af374f6765 100644 --- a/ietf/group/tests_review.py +++ b/ietf/group/tests_review.py @@ -212,13 +212,13 @@ def test_reviewer_overview(self): r = self.client.get(url) self.assertEqual(r.status_code, 200) # review team members can see reason for being unavailable - self.assertContains(r, "Availability") + self.assertContains(r, "Available") self.client.login(username="secretary", password="secretary+password") r = self.client.get(url) self.assertEqual(r.status_code, 200) # secretariat can see reason for being unavailable - self.assertContains(r, "Availability") + self.assertContains(r, "Available") # add one closed review with no response and see it is visible review_req2 = ReviewRequestFactory(state_id='completed',team=team) diff --git a/ietf/templates/group/reviewer_overview.html b/ietf/templates/group/reviewer_overview.html index 39ba7606a9..b4b1b6e3d7 100644 --- a/ietf/templates/group/reviewer_overview.html +++ b/ietf/templates/group/reviewer_overview.html @@ -24,7 +24,7 @@
+
Has already been assigned a document within the given interval.
@@ -44,89 +44,102 @@
Next | +Next | Reviewer | -Recent history | -Days since completed | -Settings | +
+
+
+ Assigned
+ Deadline
+ State
+ Days for review
+ Document
+ |
+ Days since review | +Settings | |||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
{{ forloop.counter }} | +
- {% person_link person %}
- {% if person.settings_url %}
- Edit
-
- {% endif %}
+ {% person_link person with_email=False %}
+
+
+ {% if person.settings_url %}
+
+
+
+ {% endif %}
+
+
|
+
{% if person.latest_reqs %}
-
+
+ {% endfor %}
{% endif %}
+ {{ assigned_time|date|split:"-"|join:"-
+
+
+ {{ state.name }}
+
+
+ {% if assignment_to_closure_days != None %}
+ {{ assignment_to_closure_days }}
+ {% endif %}
+
+
+ {{ doc_name }}{% if reviewed_rev %}-{{ reviewed_rev }}{% endif %}
+
+ |
- + + | {% if person.days_since_completed_review != 9999 %} {{ person.days_since_completed_review }} {% endif %} | -
- {% if person.settings.min_interval %}
- {{ person.settings.get_min_interval_display }}
- - {% endif %} - {% if person.settings.skip_next %} - Skip: {{ person.settings.skip_next }} - - {% endif %} - {% if person.settings.filter_re %} - Filter: {{ person.settings.filter_re|truncatechars:15 }}
- - {% endif %} - {% if person.unavailable_periods %} - {% include "review/unavailable_table.html" with unavailable_periods=person.unavailable_periods %} - {% endif %} + + | + {% include "review/unavailable_table.html" with person=person unavailable_periods=person.unavailable_periods %} + | +- {{ r.state.name }} + {{ r.state.name }} |
{% if r.result %}{{ r.result.name }}{% endif %}
@@ -186,10 +186,10 @@
|