Merge pull request arachnys#562 from iPalash/develop

Option to enable/disable subscription & duty pages
saruberoz · Sep 15, 2017 · edba32a · edba32a
2 parents 3e2e1d4 + 3ae07bb
commit edba32a
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 5 deletions.
diff --git a/cabot/cabotapp/views.py b/cabot/cabotapp/views.py
@@ -296,7 +296,10 @@ class Meta:
                 'data-rel': 'chosen',
                 'style': 'width: 70%',
             }),
-            'users_to_notify': forms.CheckboxSelectMultiple(),
+            'users_to_notify': forms.SelectMultiple(attrs={
+                'data-rel': 'chosen',
+                'style': 'width: 70%',
+            }),
         }
 
     def __init__(self, *args, **kwargs):
@@ -337,7 +340,10 @@ class Meta:
                 'data-rel': 'chosen',
                 'style': 'width: 70%',
             }),
-            'users_to_notify': forms.CheckboxSelectMultiple(),
+            'users_to_notify': forms.SelectMultiple(attrs={
+                'data-rel': 'chosen',
+                'style': 'width: 70%',
+            }),
             'hackpad_id': forms.TextInput(attrs={'style': 'width:70%;'}),
             'runbook_link': forms.TextInput(attrs={'style': 'width:70%;'}),
         }

diff --git a/cabot/context_processors.py b/cabot/context_processors.py
@@ -0,0 +1,8 @@
+from django.conf import settings
+
+
+def global_settings(request):
+    return {
+        'ENABLE_SUBSCRIPTION': settings.ENABLE_SUBSCRIPTION,
+        'ENABLE_DUTY_ROTA': settings.ENABLE_DUTY_ROTA,
+    }
diff --git a/cabot/settings.py b/cabot/settings.py
@@ -112,6 +112,7 @@
             'django.template.context_processors.request',
             'django.contrib.auth.context_processors.auth',
             'django.contrib.messages.context_processors.messages',
+            'cabot.context_processors.global_settings',
         ],
         'debug': force_bool(os.environ.get('TEMPLATE_DEBUG', False))
     },
@@ -307,3 +308,5 @@
     SOCIAL_AUTH_GOOGLE_OAUTH2_WHITELISTED_DOMAINS = os.environ.get('AUTH_GOOGLE_OAUTH2_WHITELISTED_DOMAINS', '').split(',')
 
 EXPOSE_USER_API = force_bool(os.environ.get('EXPOSE_USER_API', False))
+ENABLE_SUBSCRIPTION = force_bool(os.environ.get('ENABLE_SUBSCRIPTION', True))
+ENABLE_DUTY_ROTA = force_bool(os.environ.get('ENABLE_DUTY_ROTA', True))
diff --git a/cabot/templates/base.html b/cabot/templates/base.html
@@ -44,10 +44,10 @@
     </ul>
   </ul>
   <ul class="nav navbar-nav navbar-right">
-          <li>
+          <li class="{% if not ENABLE_SUBSCRIPTION%}hidden{%endif%}">
             <a href="{% url 'subscriptions' %}"><i class="fa fa-table"></i> Alert subscriptions</a>
           </li>
-          <li>
+          <li class="{% if not ENABLE_DUTY_ROTA%}hidden{%endif%}">
             <a href="{% url 'shifts' %}"><i class="glyphicon glyphicon-time"></i> Duty rota</a>
           </li>
           <li class="dropdown">
@@ -73,4 +73,4 @@
             </ul>
           </li>
         </ul>
-{% endblock header_navbar_menu %}
+{% endblock header_navbar_menu %}