From 990e626631c52b70d37378f71eab5c1c11031b14 Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Wed, 2 Mar 2016 21:19:02 +0000 Subject: [PATCH] Remove tick banner MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit I’d like to see if we can get away without this now that the page updates. In research the ‘We’ve started’ part of the message confused people, especially when they’d only sent one message. --- app/main/views/send.py | 1 - tests/app/main/views/test_send.py | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/app/main/views/send.py b/app/main/views/send.py index 8f7aa93e9..da7bc6ba6 100644 --- a/app/main/views/send.py +++ b/app/main/views/send.py @@ -230,7 +230,6 @@ def check_messages(service_id, upload_id): else: raise e - flash('We’ve started sending your messages', 'default_with_tick') return redirect( url_for('main.view_job', service_id=service_id, job_id=upload_id) ) diff --git a/tests/app/main/views/test_send.py b/tests/app/main/views/test_send.py index 7e8b9e087..5404291d8 100644 --- a/tests/app/main/views/test_send.py +++ b/tests/app/main/views/test_send.py @@ -265,5 +265,5 @@ def test_create_job_should_call_api( response = client.post(url, data=job_data, follow_redirects=True) assert response.status_code == 200 - assert 'We’ve started sending your messages' in response.get_data(as_text=True) + assert original_file_name in response.get_data(as_text=True) mock_create_job.assert_called_with(job_id, service_id, template_id, original_file_name, notification_count)