From 6030cccb5dd124e57fcff9331db180d16ef1cd44 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Sun, 3 Apr 2016 12:13:29 +0100
Subject: [PATCH] =?UTF-8?q?Remove=20the=20=E2=80=98next=20step=E2=80=99=20?=
 =?UTF-8?q?banner=20from=20choose=20template?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This page should be obvious enough that this banner isn’t needed. Will
have to keep an eye on this.
---
 app/main/views/send.py                   |  2 --
 app/templates/views/choose-template.html | 11 -----------
 2 files changed, 13 deletions(-)

diff --git a/app/main/views/send.py b/app/main/views/send.py
index 6b1fe2618..5ce29476c 100644
--- a/app/main/views/send.py
+++ b/app/main/views/send.py
@@ -76,7 +76,6 @@ def choose_template(service_id, template_type):
 
     if template_type not in ['email', 'sms']:
         abort(404)
-    jobs = job_api_client.get_job(service_id)['data']
 
     return render_template(
         'views/choose-template.html',
@@ -90,7 +89,6 @@ def choose_template(service_id, template_type):
         template_type=template_type,
         page_heading=get_page_headings(template_type),
         service=service,
-        has_jobs=len(jobs),
         service_id=service_id
     )
 
diff --git a/app/templates/views/choose-template.html b/app/templates/views/choose-template.html
index 2d0b56551..4e1a0dcbf 100644
--- a/app/templates/views/choose-template.html
+++ b/app/templates/views/choose-template.html
@@ -37,17 +37,6 @@
       {% endif %}
     </div>
 
-    {% if not has_jobs %}
-      {% if current_user.has_permissions(permissions=['send_texts', 'send_emails', 'send_letters'], any_=True) %}
-      {{ banner(
-        """
-          Send yourself a test
-        """,
-        subhead='Next step:',
-        type="tip"
-      )}}
-      {% endif %}
-    {% endif %}
     <div class="grid-row">
       {% for template in templates %}
         <div class="column-two-thirds">