Add personal review overview page to the menu

 - Legacy-Id: 12083

Author: OleLaursen

ietf/ietfauth/utils.py

@@ -69,13 +69,14 @@ def has_role(user, role_names, *args, **kwargs):
             "Nomcom": Q(person=person, group__type="nomcom", group__acronym__icontains=kwargs.get('year', '0000')),
             "Liaison Manager": Q(person=person,name="liaiman",group__type="sdo",group__state="active", ),
             "Authorized Individual": Q(person=person,name="auth",group__type="sdo",group__state="active", ),
+            "Reviewer": Q(person=person, name="reviewer", group__state="active"),
             }
 
         filter_expr = Q()
         for r in role_names:
             filter_expr |= role_qs[r]
 
-        user.roles_check_cache[key] = bool(Role.objects.filter(filter_expr)[:1])
+        user.roles_check_cache[key] = bool(Role.objects.filter(filter_expr).exists())
 
     return user.roles_check_cache[key]
 

ietf/templates/base/menu_user.html

@@ -25,6 +25,10 @@
   <li><a href="{% url "ietf.ietfauth.views.create_account" %}">{% if request.user.is_authenticated %}Manage account{% else %}New account{% endif %}</a></li>
   <li><a href="{%url "ietf.cookies.views.preferences" %}" rel="nofollow">Preferences</a></li>
 
+  {% if user|has_role:"Reviewer" %}
+    <li><a href="{% url "ietf.ietfauth.views.review_overview" %}">My reviews</a></li>
+  {% endif %}
+
   {% if user|has_role:"Area Director" %}
     {% if flavor == "top" %}<li class="divider hidden-xs"></li>{% endif %}
     <li {%if flavor == "top" %}class="dropdown-header hidden-xs"{% else %}class="nav-header"{% endif %}>AD dashboard</li>

ietf/templates/ietfauth/review_overview.html

@@ -46,7 +46,7 @@ <h2>Assigned reviews</h2>
   {% endif %}
 
 
-  <h2>Latest closed requests</h2>
+  <h2>Latest closed review requests</h2>
 
   {% if closed_review_requests %}
     <table class="table table-condensed table-striped">
@@ -77,7 +77,7 @@ <h2>Latest closed requests</h2>
       </tbody>
     </table>
   {% else %}
-    <p>Did not find any closed requests assigned to you.</p>
+    <p>Did not find any closed review requests assigned to you.</p>
   {% endif %}
 
 
@@ -134,7 +134,7 @@ <h2>Settings for {{ t }}</h2>
       </tr>
       <tr>
         <th>Filter regexp</th>
-        <td>{{ t.reviewer_settings.filter_re|default:"(None)" }}</td>
+        <td><code>{{ t.reviewer_settings.filter_re|default:"(None)" }}</code></td>
       </tr>
       <tr>
         <th>Unavailable periods</th>