Bug 19179

Summary: Email option for SMSSendDriver is not documented as a valid setting
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: System AdministrationAssignee: Charlotte Cordwell <charlotte.cordwell123>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: trivial    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, m.de.rooy, magnus, martin.renvoize, nick, roch.damour
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18091
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 9021    
Bug Blocks:    
Attachments: Bug 19179: Email option for SMSSendDriver is not documented as a valid setting 1) Apply patch. 2) Go to Administration --> search SMSSendDriver 3) Check if this message is at th end of the box:"If you would prefer to receive SMS via E-mail set SMSSendDriv
Bug 19179: Email option for SMSSendDriver is not documented as a valid setting
Bug 19179: Email option for SMSSendDriver is not documented
Bug 19179: Email option for SMSSendDriver is not documented
Bug 19179: Email option for SMSSendDriver is not documented
Bug 19179: (QA follow-up) Replace second E-mail

Description Katrin Fischer 2017-08-26 15:24:43 UTC
The system preference SMSSendDriver doesn't mention the option to use Email in order to activate the feature for sending messages as sms using email gateways.

See: http://bywatersolutions.com/2016/02/26/koha-sms/
Comment 1 Charlotte Cordwell 2018-01-16 03:08:44 UTC
Created attachment 70538 [details] [review]
Bug 19179: Email option for SMSSendDriver is not documented as a valid setting 1) Apply patch. 2) Go to Administration --> search SMSSendDriver 3) Check if this message is at th end of the box:"If you would prefer to receive SMS via E-mail set SMSSendDriv
Comment 2 Charlotte Cordwell 2018-01-16 03:10:05 UTC
Created attachment 70539 [details] [review]
Bug 19179: Email option for SMSSendDriver is not documented as a valid setting

1) Apply patch.
2) Go to Administration --> search SMSSendDriver
3) Check if this message is at th end of the box:"If you would prefer to receive SMS via E-mail set SMSSendDriver to E-mail "
Comment 3 Roch D'Amour 2018-02-02 23:00:10 UTC
Could you please rebase the patch, I tried to apply it and got a SHA1 error: 
fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref).
error: could not build fake ancestor.
Thanks!
Comment 4 Mark Tompsett 2018-09-07 03:40:58 UTC
Created attachment 78498 [details] [review]
Bug 19179: Email option for SMSSendDriver is not documented

'Email' is not documented as a valid setting.

1) Apply patch.
2) Go to Administration --> search SMSSendDriver
3) Check if SMSSendDriver has this message is at the end of the box:
   "If you would prefer to receive SMS via E-mail set SMSSendDriver to Email"
Comment 5 Michal Denar 2018-09-07 11:24:53 UTC
Created attachment 78504 [details] [review]
Bug 19179: Email option for SMSSendDriver is not documented

'Email' is not documented as a valid setting.

1) Apply patch.
2) Go to Administration --> search SMSSendDriver
3) Check if SMSSendDriver has this message is at the end of the box:
   "If you would prefer to receive SMS via E-mail set SMSSendDriver to Email"

Signed-off-by:  Michal Denar <black23@gmail.coml>
Comment 6 Marcel de Rooy 2018-09-14 09:58:36 UTC
Created attachment 78645 [details] [review]
Bug 19179: Email option for SMSSendDriver is not documented

'Email' is not documented as a valid setting.

1) Apply patch.
2) Go to Administration --> search SMSSendDriver
3) Check if SMSSendDriver has this message is at the end of the box:
   "If you would prefer to receive SMS via E-mail set SMSSendDriver to Email"

Signed-off-by:  Michal Denar <black23@gmail.coml>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Marcel de Rooy 2018-09-14 09:58:42 UTC
Created attachment 78646 [details] [review]
Bug 19179: (QA follow-up) Replace second E-mail

Better write here exactly what we need to type in the pref text box.
Since E-mail will not work and Email will :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Nick Clemens 2018-09-14 23:27:48 UTC
Awesome work all!

Pushed to master for 18.11
Comment 9 Martin Renvoize 2018-09-17 13:45:54 UTC
Pushed to 18.05.x for 18.05.04
Comment 10 Fridolin Somers 2018-09-24 09:44:08 UTC
Pushed to 17.11.x for 17.11.10