You can find details of how to pay for Notify in our data sharing and financial agreement.
- Contact us to get a copy of the agreement - {% if agreement_info.agreement_signed %} - ({{ agreement_info.owner }} has already accepted it). - {% else %} - {% if agreement_info.owner and agreement_info.agreement_signed != None %} - ({{ agreement_info.owner }} hasn’t accepted it yet). - {% else %} - or find out if we already have one in place with your organisation. - {% endif %} - {% endif %} + {{ agreement_info.as_pricing_paragraph( + pricing_link=url_for('main.sign_in', next=url_for('main.pricing', _anchor='paying')), + download_link=url_for('main.agreement'), + support_link=url_for('.feedback', ticket_type='ask-question-give-feedback', body='agreement'), + signed_in=current_user.is_authenticated, + ) }}
diff --git a/app/templates/views/terms-of-use.html b/app/templates/views/terms-of-use.html index 540b7f1e1..f0576bbbd 100644 --- a/app/templates/views/terms-of-use.html +++ b/app/templates/views/terms-of-use.html @@ -22,8 +22,10 @@ Terms of use{{ agreement_info.as_terms_of_use_paragraph( + terms_link=url_for('main.sign_in', next=url_for('main.terms')), download_link=url_for('.agreement'), - contact_link=url_for('.feedback', ticket_type='ask-question-give-feedback', body='agreement-with-owner') + support_link=url_for('.feedback', ticket_type='ask-question-give-feedback', body='agreement'), + signed_in=current_user.is_authenticated )}}
diff --git a/app/utils.py b/app/utils.py index 13a0aec44..d1eb83f5c 100644 --- a/app/utils.py +++ b/app/utils.py @@ -475,23 +475,61 @@ class AgreementInfo: def as_terms_of_use_paragraph(self, **kwargs): return Markup(self._as_terms_of_use_paragraph(**kwargs)) - def _as_terms_of_use_paragraph(self, download_link, contact_link): + def _as_terms_of_use_paragraph(self, terms_link, download_link, support_link, signed_in): - if self.agreement_signed: - return ( - 'Your organisation ({}) has already accepted the ' - 'GOV.UK Notify data sharing and financial ' - 'agreement.'.format(self.owner) - ) + if not signed_in: + return (( + '{} Sign in to download a copy ' + 'or find out if one is already in place.' + ).format(self._acceptance_required, terms_link)) - if self.crown_status is not None: + if self.agreement_signed is None: + return (( + '{} Download the agreement or ' + 'contact us to find out if we already ' + 'have one in place with your organisation.' + ).format(self._acceptance_required, download_link, support_link)) + + if self.agreement_signed is False: return (( '{} Download a copy.' ).format(self._acceptance_required, download_link)) - return (( - '{} Contact us to get a copy.' - ).format(self._acceptance_required, contact_link)) + return ( + 'Your organisation ({}) has already accepted the ' + 'GOV.UK Notify data sharing and financial ' + 'agreement.'.format(self.owner) + ) + + def as_pricing_paragraph(self, **kwargs): + return Markup(self._as_pricing_paragraph(**kwargs)) + + def _as_pricing_paragraph(self, pricing_link, download_link, support_link, signed_in): + + if not signed_in: + return (( + 'Sign in to download a copy or find ' + 'out if one is already in place with your organisation.' + ).format(pricing_link)) + + if self.agreement_signed is None: + return (( + 'Download the agreement or ' + 'contact us to find out if we already ' + 'have one in place with your organisation.' + ).format(download_link, support_link)) + + return ( + 'Download the agreement ' + '({} {}).'.format( + download_link, + self.owner, + { + True: 'has already accepted it', + False: 'hasn’t accepted it yet' + }.get(self.agreement_signed) + ) + ) @property def _acceptance_required(self): @@ -508,17 +546,6 @@ class AgreementInfo: abort(404) return self.crown_status - def as_request_for_agreement(self, with_owner=False): - if with_owner and self.owner: - return ( - 'Please send me a copy of the GOV.UK Notify data sharing ' - 'and financial agreement for {} to sign.'.format(self.owner) - ) - return ( - 'Please send me a copy of the GOV.UK Notify data sharing ' - 'and financial agreement.' - ) - @staticmethod def get_matching_function(email_address_or_domain): diff --git a/tests/app/main/views/test_feedback.py b/tests/app/main/views/test_feedback.py index 63aba1e40..87f6cf4d6 100644 --- a/tests/app/main/views/test_feedback.py +++ b/tests/app/main/views/test_feedback.py @@ -88,16 +88,8 @@ def test_get_feedback_page(client, ticket_type, expected_status_code): ( 'agreement', ( - 'Please send me a copy of the GOV.UK Notify data sharing ' - 'and financial agreement.' - ) - ), - ( - 'agreement-with-owner', - ( - 'Please send me a copy of the GOV.UK Notify data sharing ' - 'and financial agreement for Marine Management ' - 'Organisation to sign.' + 'Please can you tell me if there’s an agreement in place ' + 'between GOV.UK Notify and my organisation?' ) ), ( diff --git a/tests/app/main/views/test_index.py b/tests/app/main/views/test_index.py index fc4fe1bd4..be6348ac3 100644 --- a/tests/app/main/views/test_index.py +++ b/tests/app/main/views/test_index.py @@ -101,7 +101,25 @@ def test_terms_is_generic_if_user_is_not_logged_in( assert normalize_spaces(page.select('main p')[1].text) == ( 'Your organisation must also accept our data sharing and ' - 'financial agreement. Contact us to get a copy.' + 'financial agreement. Sign in to download a copy or find out ' + 'if one is already in place.' + ) + + +def test_pricing_is_generic_if_user_is_not_logged_in( + client +): + response = client.get(url_for('main.pricing')) + assert response.status_code == 200 + page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser') + last_paragraph = page.select('main p')[-1] + assert normalize_spaces(last_paragraph.text) == ( + 'Sign in to download a copy or find out if one is already ' + 'in place with your organisation.' + ) + assert last_paragraph.select_one('a')['href'] == url_for( + 'main.sign_in', + next=url_for('main.pricing', _anchor='paying'), ) @@ -119,7 +137,7 @@ def test_terms_is_generic_if_user_is_not_logged_in( ), None, ( - 'Contact us to get a copy of the agreement ' + 'Download the agreement ' '(Cabinet Office has already accepted it).' ), ), @@ -135,7 +153,7 @@ def test_terms_is_generic_if_user_is_not_logged_in( 'main.agreement', ), ( - 'Contact us to get a copy of the agreement ' + 'Download the agreement ' '(Aylesbury Town Council hasn’t accepted it yet).' ), ), @@ -143,16 +161,16 @@ def test_terms_is_generic_if_user_is_not_logged_in( 'larry@downing-street.gov.uk', ( 'Your organisation must also accept our data sharing and ' - 'financial agreement. Contact us to get a copy.' + 'financial agreement. Download the agreement or contact us ' + 'to find out if we already have one in place with your ' + 'organisation.' ), partial( url_for, - 'main.feedback', - ticket_type='ask-question-give-feedback', - body='agreement-with-owner', + 'main.agreement', ), ( - 'Contact us to get a copy of the agreement or find out if ' + 'Download the agreement or contact us to find out if ' 'we already have one in place with your organisation.' ), ), @@ -167,8 +185,8 @@ def test_terms_is_generic_if_user_is_not_logged_in( 'main.agreement', ), ( - 'Contact us to get a copy of the agreement (Met Office ' - 'hasn’t accepted it yet).' + 'Download the agreement (Met Office hasn’t accepted it ' + 'yet).' ), ), ])