Moved email open assignment summary from subpage to the main review request page. Fixes ietf-tools#2091. Commit ready for merge

 - Legacy-Id: 14356

Author: kivinen

ietf/group/utils.py

@@ -221,7 +221,7 @@ def construct_group_menu_context(request, group, selected, group_type, others):
 
         if Role.objects.filter(name="secr", group=group, person__user=request.user).exists():
             actions.append((u"Secretary settings", urlreverse(ietf.group.views.change_review_secretary_settings, kwargs=kwargs)))
-
+            actions.append((u"Email open assignments summary", urlreverse(ietf.group.views.email_open_review_assignments, kwargs=dict(acronym=group.acronym, group_type=group.type_id))))
 
     if group.state_id != "conclude" and (is_admin or can_manage):
         can_edit_group = True

ietf/templates/group/manage_review_requests.html

@@ -15,15 +15,6 @@
 
   <h1>Manage {{ assignment_status }} open review requests for {{ group.acronym }}</h1>
 
-  <p>Other options:
-    <a href="{% url "ietf.group.views.review_requests" group_type=group.type_id acronym=group.acronym %}">All review requests</a>
-    - <a href="{% url "ietf.group.views.reviewer_overview" group_type=group.type_id acronym=group.acronym %}">Reviewers</a>
-    - <a href="{% url "ietf.group.views.email_open_review_assignments" group_type=group.type_id acronym=group.acronym  %}?next={{ request.get_full_path|urlencode }}">Email open assignments summary</a>
-    {% if other_assignment_status %}
-      - <a href="{% url "ietf.group.views.manage_review_requests" group_type=group.type_id acronym=group.acronym assignment_status=other_assignment_status %}">Manage {{ other_assignment_status }} reviews</a>
-    {% endif %}
-  </p>
-
   {% if newly_closed > 0 or newly_opened > 0 or newly_assigned > 0 %}
     <p class="alert alert-danger">
       Changes since last refresh: