For more information, you can download a copy of the agreement (PDF). -
The agreement is confidential and should not be shared outside your organisation.
diff --git a/app/templates/views/agreement/service-agreement.html b/app/templates/views/agreement/service-agreement.html index 91d762fe3..d9465fd49 100644 --- a/app/templates/views/agreement/service-agreement.html +++ b/app/templates/views/agreement/service-agreement.html @@ -27,9 +27,6 @@Once accepted, the agreement covers all Notify services from {{ current_service.organisation.name }}.
-- Download a copy of the data sharing and financial agreement (PDF). -
The agreement is confidential and should not be shared outside your organisation.
diff --git a/app/templates/views/organisations/organisation/billing.html b/app/templates/views/organisations/organisation/billing.html index d48bb9b0e..96f15cb7a 100644 --- a/app/templates/views/organisations/organisation/billing.html +++ b/app/templates/views/organisations/organisation/billing.html @@ -22,20 +22,10 @@ {{ current_org.agreement_signed_on_behalf_of_name or current_org.agreement_signed_by.name }} signed the agreement on behalf of {{ current_org.name}}. -- Download the current version of the agreement (PDF) - -
{% elif current_org.agreement_signed %}{{ current_org.name}} has accepted the US Notify data sharing and financial agreement.
-- Download the current version of the agreement (PDF) - -
{% elif current_org.agreement_signed is false %}{{ current_org.name}} needs to accept the US Notify data sharing and financial agreement. diff --git a/manifest.yml b/manifest.yml index 0d437cc6e..2bd475250 100644 --- a/manifest.yml +++ b/manifest.yml @@ -17,7 +17,6 @@ applications: - notifications-api-csv-upload-bucket-((env)) - notifications-api-contact-list-bucket-((env)) - notifications-admin-logo-upload-bucket-((env)) - - notifications-admin-mou-upload-bucket-((env)) env: NOTIFY_APP_NAME: admin diff --git a/terraform/staging/main.tf b/terraform/staging/main.tf index 73946a736..f60fb31bd 100644 --- a/terraform/staging/main.tf +++ b/terraform/staging/main.tf @@ -29,14 +29,3 @@ module "logo_upload_bucket" { recursive_delete = local.recursive_delete s3_service_name = "${local.app_name}-logo-upload-bucket-${local.env}" } - -module "mou_upload_bucket" { - source = "github.com/18f/terraform-cloudgov//s3" - - cf_user = var.cf_user - cf_password = var.cf_password - cf_org_name = local.cf_org_name - cf_space_name = local.cf_space_name - recursive_delete = local.recursive_delete - s3_service_name = "${local.app_name}-mou-upload-bucket-${local.env}" -} diff --git a/tests/app/main/views/organisations/test_organisations.py b/tests/app/main/views/organisations/test_organisations.py index 331d83cd8..598948506 100644 --- a/tests/app/main/views/organisations/test_organisations.py +++ b/tests/app/main/views/organisations/test_organisations.py @@ -1669,7 +1669,7 @@ def test_organisation_billing_page_when_the_agreement_is_signed_by_a_known_perso assert '2.5 of the US Notify data sharing and financial agreement on 20 February 2020' in normalize_spaces( page.text) assert f'{expected_signatory} signed' in page.text - assert page.select_one('main a')['href'] == url_for('.organisation_download_agreement', org_id=ORGANISATION_ID) + # assert page.select_one('main a')['href'] == url_for('.organisation_download_agreement', org_id=ORGANISATION_ID) def test_organisation_billing_page_when_the_agreement_is_signed_by_an_unknown_person( @@ -1690,7 +1690,7 @@ def test_organisation_billing_page_when_the_agreement_is_signed_by_an_unknown_pe assert page.h1.string == 'Billing' assert (f'{organisation_one["name"]} has accepted the US Notify data ' 'sharing and financial agreement.') in page.text - assert page.select_one('main a')['href'] == url_for('.organisation_download_agreement', org_id=ORGANISATION_ID) + # assert page.select_one('main a')['href'] == url_for('.organisation_download_agreement', org_id=ORGANISATION_ID) @pytest.mark.parametrize('agreement_signed, expected_content', [ @@ -1719,14 +1719,14 @@ def test_organisation_billing_page_when_the_agreement_is_not_signed( @pytest.mark.parametrize('crown, expected_status, expected_file_fetched, expected_file_served', ( - ( - True, 200, 'crown.pdf', - 'US Notify data sharing and financial agreement.pdf', - ), - ( - False, 200, 'non-crown.pdf', - 'US Notify data sharing and financial agreement (non-crown).pdf', - ), + # ( + # True, 200, 'crown.pdf', + # 'US Notify data sharing and financial agreement.pdf', + # ), + # ( + # False, 200, 'non-crown.pdf', + # 'US Notify data sharing and financial agreement (non-crown).pdf', + # ), ( None, 404, None, None, diff --git a/tests/app/main/views/test_agreement.py b/tests/app/main/views/test_agreement.py index ef666a921..0744656a3 100644 --- a/tests/app/main/views/test_agreement.py +++ b/tests/app/main/views/test_agreement.py @@ -23,21 +23,12 @@ class MockS3Object(): ( True, True, partial(url_for, 'main.request_to_go_live', service_id=SERVICE_ONE_ID), - [ - ( - ['govuk-link', 'govuk-link--no-visited-state'], - partial(url_for, 'main.service_download_agreement', service_id=SERVICE_ONE_ID), - ), - ] + [] ), ( False, False, partial(url_for, 'main.request_to_go_live', service_id=SERVICE_ONE_ID), [ - ( - ['govuk-link', 'govuk-link--no-visited-state'], - partial(url_for, 'main.service_download_agreement', service_id=SERVICE_ONE_ID), - ), ( ['govuk-button'], partial(url_for, 'main.service_accept_agreement', service_id=SERVICE_ONE_ID), @@ -48,10 +39,6 @@ class MockS3Object(): False, True, partial(url_for, 'main.request_to_go_live', service_id=SERVICE_ONE_ID), [ - ( - ['govuk-link', 'govuk-link--no-visited-state'], - partial(url_for, 'main.service_download_agreement', service_id=SERVICE_ONE_ID), - ), ( ['govuk-button'], partial(url_for, 'main.service_accept_agreement', service_id=SERVICE_ONE_ID), @@ -126,14 +113,14 @@ def test_unknown_gps_and_trusts_are_redirected( @pytest.mark.parametrize('crown, expected_status, expected_file_fetched, expected_file_served', ( - ( - True, 200, 'crown.pdf', - 'US Notify data sharing and financial agreement.pdf', - ), - ( - False, 200, 'non-crown.pdf', - 'US Notify data sharing and financial agreement (non-crown).pdf', - ), + # ( + # True, 200, 'crown.pdf', + # 'US Notify data sharing and financial agreement.pdf', + # ), + # ( + # False, 200, 'non-crown.pdf', + # 'US Notify data sharing and financial agreement (non-crown).pdf', + # ), ( None, 404, None, None, @@ -490,8 +477,8 @@ def test_confirm_agreement_page_persists( 'main.public_download_agreement', )) @pytest.mark.parametrize('variant, expected_status', ( - ('crown', 200), - ('non-crown', 200), + # ('crown', 200), + # ('non-crown', 200), ('foo', 404), )) def test_show_public_agreement_page(