Merge pull request #2762 from alphagov/letter-logo-reword

Finesse wording on letter branding request page
This commit is contained in:
Chris Hill-Scott
2019-02-13 15:59:02 +00:00
committed by GitHub
2 changed files with 17 additions and 11 deletions

View File

@@ -11,17 +11,23 @@
<h1 class="heading-large">Letter branding</h1>
<div class="grid-row">
<div class="column-three-quarters">
<p>
{% if current_service.letter_branding_id %}
{% if current_service.letter_branding_id %}
<p>
Your letters have the {{ current_service.letter_branding.name }} logo.
{% else %}
Your letters have no logo.
{% endif %}
</p>
<p>
<a href="{{ url_for('main.feedback', ticket_type='ask-question-give-feedback', body='letter-branding') }}">Contact support</a>
if you want to use a different logo.
</p>
</p>
<p>
<a href="{{ url_for('main.feedback', ticket_type='ask-question-give-feedback', body='letter-branding') }}">Contact support</a>
if you want to use a different logo.
</p>
{% else %}
<p>
Your letters do not have a logo.
</p>
<p>
<a href="{{ url_for('main.feedback', ticket_type='ask-question-give-feedback', body='letter-branding') }}">Contact support</a>
if you want to add your organisations logo.
</p>
{% endif %}
{% if from_template %}
{{ page_footer(
back_link=url_for('.view_template', service_id=current_service.id, template_id=from_template),

View File

@@ -2089,7 +2089,7 @@ def test_request_letter_branding(
service_id=SERVICE_ONE_ID,
**extra_args
)
assert request_page.select_one('main p').text.strip() == 'Your letters have no logo.'
assert request_page.select_one('main p').text.strip() == 'Your letters do not have a logo.'
link_href = request_page.select_one('main a')['href']
feedback_page = client_request.get_url(link_href)
assert feedback_page.select_one('textarea').text.strip() == (