darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-09 02:19:31 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3099 from alphagov/update-receive-text-message-content

Browse Source 
Update explanation of inbound SMS
This commit is contained in:
karlchillmaid
2019-08-23 14:13:54 +01:00
committed by GitHub
parent 86e2b4b117 922a7fc9dc
commit 0d7005094a
2 changed files with 20 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
app/templates/views/service-settings/set-inbound-sms.html
View File

@@ -19,6 +19,9 @@
<p>
Your service can receive text messages sent to {{ current_service.inbound_number }}.
</p>
<p>
You can still send text messages from a sender name if you need to, but users will not be able to reply to those messages.
</p>
<p>
<a href="{{ url_for('.support') }}">Contact us</a> if you want to switch this feature off.
</p>
@@ -33,9 +36,18 @@
<a href="{{ url_for('.support') }}">Contact us</a> if you want to be able to receive text messages from your users.
</p>
<p>
We’ll create a special phone number for them to contact. You'll be able to see
the messages on your dashboard, or get them using the API.
We’ll create a unique phone number for your service that they can reply to.
</p>
<p>
Text messages you send will come from this phone number instead of your sender name (currently {{ current_service.default_sms_sender }}).
</p>
<p>
You can still send text messages from a sender name if you need to, but users will not be able to reply to those messages.
</p>
<p>
You can see and reply to the messages you receive when you sign in to Notify, or get them using the API.
</p>
<p>
It does not cost you anything to receive text messages. Users will pay their standard text message rate.
</p>
{% endif %}

6
tests/app/main/views/test_service_settings.py
View File

@@ -4100,6 +4100,7 @@ def test_invitation_pages(
client_request,
service_one,
mock_get_inbound_number_for_service,
single_sms_sender,
endpoint,
permissions,
expected_p,
@@ -4138,6 +4139,7 @@ def test_set_inbound_sms_when_inbound_number_is_not_set(
service_one,
single_reply_to_email_address,
single_letter_contact_block,
single_sms_sender,
mocker,
mock_get_all_letter_branding,
):
@@ -4152,11 +4154,15 @@ def test_set_inbound_sms_when_inbound_number_is_not_set(
@pytest.mark.parametrize('user, expected_paragraphs', [
(active_user_with_permissions, [
'Your service can receive text messages sent to 07700900123.',
'You can still send text messages from a sender name if you '
'need to, but users will not be able to reply to those messages.',
'Contact us if you want to switch this feature off.',
'You can set up callbacks for received text messages on the API integration page.',
]),
(active_user_no_api_key_permission, [
'Your service can receive text messages sent to 07700900123.',
'You can still send text messages from a sender name if you '
'need to, but users will not be able to reply to those messages.',
'Contact us if you want to switch this feature off.',
]),
])