mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-08-05 22:21:34 -04:00
Remove tick banner
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.
This commit is contained in:
@@ -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)
|
||||
)
|
||||
|
||||
@@ -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)
|
||||
|
||||
Reference in New Issue
Block a user