| Summary: | Send MEMBERSHIP_EXPIRY notice by SMS | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Michal Denar <black23> |
| Component: | Notices | Assignee: | Julian Maurice <julian.maurice> |
| Status: | Needs documenting --- | QA Contact: | Kyle M Hall (khall) <kyle> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | black23, dcook, josef.moravec, julian.maurice, kyle, matt.blenkinsop, rbit |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | Sponsored | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
23.05.00
|
|
| Circulation function: | |||
| Attachments: |
Bug 23773: Send membership expiry notices by sms too
Bug 23773: Send membership expiry notices by sms too Bug 23773: Send membership expiry notices by sms too |
||
|
Description
Michal Denar
2019-10-08 08:20:37 UTC
I'm working on that. Marking as assigned Created attachment 148123 [details] [review] Bug 23773: Send membership expiry notices by sms too Test plan: 1. Verify in "About" page that SMS::Send is installed. If not, install it. 2. Set system preferences: - SMSSendDriver = 'Test' - MembershipExpiryDaysNotice = 30 3. In "Notices and Slips" tool, edit the 'MEMBERSHIP_EXPIRY' letter and verify that both email and sms parts are filled. Use a different title and body for the sms part 4. Create a new borrower with an email address and an SMS number. Set their expiry date to a date next week 5. Run misc/cronjobs/membership_expiry.pl --confirm 6. Verify in the patron "Notices" tab that two notices are pending, one for email, one for sms Sponsored-by: Médiathèque de Montauban Created attachment 148244 [details] [review] Bug 23773: Send membership expiry notices by sms too Test plan: 1. Verify in "About" page that SMS::Send is installed. If not, install it. 2. Set system preferences: - SMSSendDriver = 'Test' - MembershipExpiryDaysNotice = 30 3. In "Notices and Slips" tool, edit the 'MEMBERSHIP_EXPIRY' letter and verify that both email and sms parts are filled. Use a different title and body for the sms part 4. Create a new borrower with an email address and an SMS number. Set their expiry date to a date next week 5. Run misc/cronjobs/membership_expiry.pl --confirm 6. Verify in the patron "Notices" tab that two notices are pending, one for email, one for sms Sponsored-by: Médiathèque de Montauban Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> I've had a library ask for this and I thought it sounded familiar... I'll make a note to come back to this when I have more time. Created attachment 150406 [details] [review] Bug 23773: Send membership expiry notices by sms too Test plan: 1. Verify in "About" page that SMS::Send is installed. If not, install it. 2. Set system preferences: - SMSSendDriver = 'Test' - MembershipExpiryDaysNotice = 30 3. In "Notices and Slips" tool, edit the 'MEMBERSHIP_EXPIRY' letter and verify that both email and sms parts are filled. Use a different title and body for the sms part 4. Create a new borrower with an email address and an SMS number. Set their expiry date to a date next week 5. Run misc/cronjobs/membership_expiry.pl --confirm 6. Verify in the patron "Notices" tab that two notices are pending, one for email, one for sms Sponsored-by: Médiathèque de Montauban Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> QA note: My initial reaction was to require a cli switch, but simply not having that notice will disable the feature from the UI, which is preferable. Curious if this one will make it into 23.05 Pushed to master for 23.05. Nice work everyone, thanks! Enhancement - not backporting to 22.11.x |