altf4arnold/datatracker
1
0
Fork 0
Code Issues Pull requests Actions 3 Packages Projects Releases Wiki Activity

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

Browse source 
- Legacy-Id: 14356
This commit is contained in:
Tero Kivinen 2017-11-11 06:29:05 +00:00
parent 5ac6cf3987
commit 7c7e9ae698
2 changed files with 1 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ietf/group/utils.py
View file

@ -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

9
ietf/templates/group/manage_review_requests.html
View file

@ -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: