From 6ec9a25dd15549dfdbc8ca8c07066b1a3f419a32 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Wed, 8 Jul 2020 11:34:16 +0100
Subject: [PATCH] Add a dashboard for broadcast services

This is just a placeholder for now
---
 app/main/views/broadcast.py                  |  9 +++++++++
 app/navigation.py                            |  6 ++++++
 app/templates/main_nav.html                  |  4 +++-
 app/templates/views/broadcast/dashboard.html | 15 +++++++++++++++
 tests/app/main/views/test_broadcast.py       | 12 ++++++++++++
 5 files changed, 45 insertions(+), 1 deletion(-)
 create mode 100644 app/templates/views/broadcast/dashboard.html

diff --git a/app/main/views/broadcast.py b/app/main/views/broadcast.py
index c2401e49e..7e60f3f12 100644
--- a/app/main/views/broadcast.py
+++ b/app/main/views/broadcast.py
@@ -8,6 +8,15 @@ from app.main.forms import BroadcastAreaForm, SearchByNameForm
 from app.utils import service_has_permission, user_has_permissions
 
 
+@main.route('/services/<uuid:service_id>/broadcast-dashboard')
+@user_has_permissions()
+@service_has_permission('broadcast')
+def broadcast_dashboard(service_id):
+    return render_template(
+        'views/broadcast/dashboard.html',
+    )
+
+
 @main.route('/services/<uuid:service_id>/broadcast')
 @user_has_permissions('send_messages')
 @service_has_permission('broadcast')
diff --git a/app/navigation.py b/app/navigation.py
index 07458080c..d5c90b836 100644
--- a/app/navigation.py
+++ b/app/navigation.py
@@ -351,6 +351,7 @@ class HeaderNavigation(Navigation):
         'old_guest_list',
         'who_can_use_notify',
         'who_its_for',
+        'broadcast_dashboard',
         'broadcast',
         'preview_broadcast_areas',
         'choose_broadcast_library',
@@ -368,6 +369,7 @@ class MainNavigation(Navigation):
 
     mapping = {
         'dashboard': {
+            'broadcast_dashboard',
             'conversation',
             'inbox',
             'monthly',
@@ -687,6 +689,9 @@ class MainNavigation(Navigation):
 class CaseworkNavigation(Navigation):
 
     mapping = {
+        'dashboard': {
+            'broadcast_dashboard',
+        },
         'send-one-off': {
             'choose_from_contact_list',
             'choose_template',
@@ -1306,6 +1311,7 @@ class OrgNavigation(Navigation):
         'old_guest_list',
         'who_can_use_notify',
         'who_its_for',
+        'broadcast_dashboard',
         'broadcast',
         'preview_broadcast_areas',
         'choose_broadcast_library',
diff --git a/app/templates/main_nav.html b/app/templates/main_nav.html
index 3e5d8b2bd..bb676343d 100644
--- a/app/templates/main_nav.html
+++ b/app/templates/main_nav.html
@@ -4,7 +4,9 @@
 <nav class="navigation">
   <ul>
   {% if current_user.has_permissions() %}
-    {% if current_user.has_permissions('view_activity') %}
+    {% if current_service.has_permission('broadcast') %}
+      <li><a class="govuk-link govuk-link--no-visited-state{{ main_navigation.is_selected('dashboard') }}" href="{{ url_for('.broadcast_dashboard', service_id=current_service.id) }}">Dashboard</a></li>
+    {% elif current_user.has_permissions('view_activity') %}
       <li><a class="govuk-link govuk-link--no-visited-state{{ main_navigation.is_selected('dashboard') }}" href="{{ url_for('.service_dashboard', service_id=current_service.id) }}">Dashboard</a></li>
     {% endif %}
     <li><a class="govuk-link govuk-link--no-visited-state{{ main_navigation.is_selected('templates') }}" href="{{ url_for('.choose_template', service_id=current_service.id) }}">Templates</a></li>
diff --git a/app/templates/views/broadcast/dashboard.html b/app/templates/views/broadcast/dashboard.html
new file mode 100644
index 000000000..c92ce3992
--- /dev/null
+++ b/app/templates/views/broadcast/dashboard.html
@@ -0,0 +1,15 @@
+{% extends "withnav_template.html" %}
+
+{% block service_page_title %}
+  Dashboard
+{% endblock %}
+
+{% block maincolumn_content %}
+
+  <h1 class="govuk-visually-hidden">Dashboard</h1>
+
+  <p class="govuk-body">
+    Broadcasts here
+  </p>
+
+{% endblock %}
diff --git a/tests/app/main/views/test_broadcast.py b/tests/app/main/views/test_broadcast.py
index 0719ae6b8..4deca0eaa 100644
--- a/tests/app/main/views/test_broadcast.py
+++ b/tests/app/main/views/test_broadcast.py
@@ -5,6 +5,7 @@ from tests.conftest import SERVICE_ONE_ID
 
 
 @pytest.mark.parametrize('endpoint, extra_args', (
+    ('.broadcast_dashboard', {}),
     ('.broadcast', {}),
     ('.preview_broadcast_areas', {}),
     ('.choose_broadcast_library', {}),
@@ -25,6 +26,17 @@ def test_broadcast_pages_403_without_permission(
     )
 
 
+def test_broadcast_dashboard(
+    client_request,
+    service_one,
+):
+    service_one['permissions'] += ['broadcast']
+    client_request.get(
+        '.broadcast_dashboard',
+        service_id=SERVICE_ONE_ID,
+    ),
+
+
 def test_broadcast_page(
     client_request,
     service_one,