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

Set min text message sender length to 4 characters

Browse Source 
We’ve learned of a change implemented today by the UK mobile
network operators, to stop allowing text message sender names
of 3 or less characters.

Adding this validation will not affect existing senders, only those
users trying to add to or update their senders.
This commit is contained in:
Chris Hill-Scott
2018-09-19 16:22:58 +01:00
parent e7aa8caf97
commit 36eac9d031
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tests/app/main/test_validators.py
View File

@@ -214,11 +214,11 @@ def test_sms_sender_form_validation(
form.validate()
assert 'Use letters and numbers only' == form.errors['sms_sender'][0]
form.sms_sender.data = '0'
form.sms_sender.data = '333'
form.validate()
assert 'Enter 3 characters or more' == form.errors['sms_sender'][0]
assert 'Enter 4 characters or more' == form.errors['sms_sender'][0]
form.sms_sender.data = '111'
form.sms_sender.data = '4444'
form.validate()
assert not form.errors