Remove prompt from dashboard

It’s no longer true to say that you need a template to create an alert.
The grey button should be the primary call to action on this page
instead.
This commit is contained in:
Chris Hill-Scott
2020-11-20 17:03:26 +00:00
parent 8f2dc72881
commit 74243a88fb
3 changed files with 10 additions and 18 deletions

View File

@@ -9,14 +9,6 @@
{% block maincolumn_content %}
{% if current_user.has_permissions('manage_templates') and not current_service.all_templates %}
<nav class="govuk-!-margin-top-2 govuk-!-margin-bottom-6">
<a class="govuk-link govuk-link--no-visited-state pill-separate-item govuk-!-padding-top-4 govuk-!-padding-bottom-4 govuk-!-font-weight-bold" href="{{ url_for('.choose_template', service_id=current_service.id) }}">
Start by creating a template
</a>
</nav>
{% endif %}
<h1 class="heading-medium">Current alerts</h1>
{{ ajax_block(

View File

@@ -20,16 +20,16 @@
{{ page_header(page_title, size='medium') }}
<p class="govuk-body">
{% if current_user.has_permissions('manage_templates') %}
You need a template before you can
{% else %}
You need to ask your service manager to add templates before you can
{% endif %}
{% if current_service.has_permission('broadcast') %}
prepare a broadcast
{%- else %}
send emails, text messages or letters
{%- endif %}.
You havent added any templates yet.
{% else %}
{% if current_user.has_permissions('manage_templates') %}
You need a template before you can
{% else %}
You need to ask your service manager to add templates before you can
{% endif %}
send emails, text messages or letters.
{% endif %}
</p>
{% else %}

View File

@@ -43,7 +43,7 @@ from tests.conftest import (
'You need a template before you can send emails, text messages or letters.'
)),
(['broadcast'], (
'You need a template before you can prepare a broadcast.'
'You havent added any templates yet.'
)),
))
def test_should_show_empty_page_when_no_templates(