darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-23 07:46:23 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #796 from alphagov/hide-csv-on-dashboard-again

Browse Source 
Really hide the ‘download CSV’ link in the tour
This commit is contained in:
Chris Hill-Scott
2016-07-19 13:20:50 +01:00
committed by GitHub
parent 6d5561267b 83c1035f1b
commit b378e8aebd
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/main/views/jobs.py
View File

@@ -97,7 +97,8 @@ def view_job(service_id, job_id):
".view_job_updates", ".view_job_updates",
service_id=service_id, service_id=service_id,
job_id=job['id'], job_id=job['id'],
status=request.args.get('status', '') status=request.args.get('status', ''),
help=get_help_argument()
), ),
partials=get_job_partials(job), partials=get_job_partials(job),
help=get_help_argument() help=get_help_argument()

7
tests/app/main/views/test_jobs.py
View File

@@ -120,6 +120,13 @@ def test_should_show_not_show_csv_download_in_tour(
)) ))
assert response.status_code == 200 assert response.status_code == 200
assert url_for(
'main.view_job_updates',
service_id=service_one['id'],
job_id=fake_uuid,
status='',
help=3
).replace('&', '&') in response.get_data(as_text=True)
assert url_for( assert url_for(
'main.view_job_csv', 'main.view_job_csv',
service_id=service_one['id'], service_id=service_one['id'],