Bug 30295

Summary: Send a notification when a recall is manually cancelled
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: CirculationAssignee: Aleisha Amohia <aleisha>
Status: Patch doesn't apply --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: andrew.auld, david, gmcharlt, kyle.m.hall, kyle, Laura.escamilla, m.de.rooy, stina.hallin
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:
Circulation function: recalls
Bug Depends on: 23781    
Bug Blocks: 29734    
Attachments: Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Use blessed patron to enqueue notice and QA fixes
Bug 30295: Use blessed patron to enqueue notice and QA fixes
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Use blessed patron to enqueue notice and QA fixes
Bug 30295: Add phone transport option
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Use blessed patron to enqueue notice and QA fixes
Bug 30295: Add phone transport option
Bug 30295: Send a notification when a recall is manually cancelled
Bug 30295: Use blessed patron to enqueue notice and QA fixes
Bug 30295: Add phone transport option

Description Aleisha Amohia 2022-03-15 01:18:01 UTC
The process would be:

1. Recall is cancelled by library staff or the requester.
2. Notification of the recall cancellation is sent via email and SMS to the recall requester.

So it would be exactly the same as the HOLD_MANUAL_CANCEL notice that we currently have for reserves, only relevant to recalls.
Comment 1 Aleisha Amohia 2022-03-16 02:00:57 UTC Comment hidden (obsolete)
Comment 2 David Nind 2022-05-11 22:39:36 UTC
Patch no longer applies:

git bz apply 30295

Bug 30295 - Send a notification when a recall is manually cancelled

131763 - Bug 30295: Send a notification when a recall is manually cancelled

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 30295: Send a notification when a recall is manually cancelled
error: sha1 information is lacking or useless (Koha/Recall.pm).
error: could not build fake ancestor
Patch failed at 0001 Bug 30295: Send a notification when a recall is manually cancelled
Comment 3 Aleisha Amohia 2022-09-07 04:54:52 UTC Comment hidden (obsolete)
Comment 4 Aleisha Amohia 2022-09-07 04:58:57 UTC Comment hidden (obsolete)
Comment 5 David Nind 2023-01-30 07:59:11 UTC
The patch no longer applies 8-(..

I'll try and test a bit quicker next time!
Comment 6 Aleisha Amohia 2023-02-22 04:00:29 UTC Comment hidden (obsolete)
Comment 7 Stina Hallin 2023-03-10 08:22:01 UTC
I could not get the cancel recall notification to show up at the notices tab. I tried different patrons and different book, also tried recalling both bib and item. I did see other recall related notices, but never a recall cancelled notice.
Comment 8 Aleisha Amohia 2023-03-13 00:16:37 UTC Comment hidden (obsolete)
Comment 9 ByWater Sandboxes 2023-04-04 13:33:26 UTC Comment hidden (obsolete)
Comment 10 Andrew Auld 2023-04-04 13:37:41 UTC
Saw both the SMS and email notice.
Confirmed that the recall cancelled notice is sent both when the recall is cancelled on the staff side and also when the recall is cancelled by the patron in the OPAC.
Comment 11 Aleisha Amohia 2023-04-30 21:21:40 UTC Comment hidden (obsolete)
Comment 12 Marcel de Rooy 2023-07-14 06:50:44 UTC
Base patch set in FQA
Comment 13 Aleisha Amohia 2023-09-07 23:15:18 UTC Comment hidden (obsolete)
Comment 14 Aleisha Amohia 2023-09-07 23:16:03 UTC
Base patch set has been unblocked.

Most recent patch is a response to changes in C4::Message->enqueue, so please retest that the notice is generated and enqueued.
Comment 15 ByWater Sandboxes 2023-09-08 06:53:56 UTC Comment hidden (obsolete)
Comment 16 Stina Hallin 2023-09-08 06:54:44 UTC
Worked as described in test plan
Comment 17 Kyle M Hall (khall) 2023-10-27 10:41:59 UTC
I think this needs to have the phone transport option as well.
Comment 18 Aleisha Amohia 2023-11-09 03:09:18 UTC Comment hidden (obsolete)
Comment 19 Aleisha Amohia 2023-11-09 03:09:21 UTC Comment hidden (obsolete)
Comment 20 Aleisha Amohia 2023-11-09 03:09:24 UTC Comment hidden (obsolete)
Comment 21 David Nind 2023-11-15 20:41:26 UTC Comment hidden (obsolete)
Comment 22 David Nind 2023-11-15 20:41:29 UTC Comment hidden (obsolete)
Comment 23 David Nind 2023-11-15 20:41:32 UTC Comment hidden (obsolete)
Comment 24 Kyle M Hall (khall) 2024-02-23 14:49:24 UTC
This needs to have support for the phone transport as well.
Comment 25 Aleisha Amohia 2024-09-26 19:21:55 UTC
Created attachment 172038 [details] [review]
Bug 30295: Send a notification when a recall is manually cancelled

This patch adds a RECALL_MANUAL_CANCEL email and sms notice which will
be sent to the requestor when their recall is cancelled. This notice
can also be configured using the messaging preferences.

To test:

1. Apply the patch, update database, restart services.
2. Ensure UseRecalls system preference is enabled and values have been
set in the circulation rules for recalls
3. Log in as a patron (Patron A) to the staff client and view your account details
4. Notice there is now a 'recall cancelled' notice that can be
configured under messaging preferences
5. Confirm this notice also shows in the messaging preferences page on
the OPAC
6. Select email and sms as a preference for the 'recall cancelled' notice and
save. You may need to enable other settings to see the sms options
(SMSSendDriver for example)
7. Check out an item to another patron (Patron B)
8. Log into the OPAC as Patron A and place a recall on the checked out
item/biblio
9. Go back to the staff client and view the biblio. Go to the recalls
tab and cancel the recall.
10. Go to Patron A's accout and view their notices tab
11. Confirm the 'recall cancelled' notice has been generated for both
email and sms.

Sponsored-by: Toi Ohomai Institute of Technology
Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com>
Signed-off-by: David Nind <david@davidnind.com>
Comment 26 Aleisha Amohia 2024-09-26 19:21:57 UTC
Created attachment 172039 [details] [review]
Bug 30295: Use blessed patron to enqueue notice and QA fixes

Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Signed-off-by: David Nind <david@davidnind.com>
Comment 27 Aleisha Amohia 2024-09-26 19:22:00 UTC
Created attachment 172041 [details] [review]
Bug 30295: Add phone transport option

Signed-off-by: David Nind <david@davidnind.com>
Comment 28 Owen Leonard 2024-12-03 14:27:46 UTC
These patches all have a signoff. Should it still be "Needs signoff"
Comment 29 David Nind 2024-12-04 20:25:52 UTC
The patches no longer apply 8-(... When that is fixed, I think the status can change to signed off.

git bz apply 30295

Bug 30295 - Send a notification when a recall is manually cancelled

172038 - Bug 30295: Send a notification when a recall is manually cancelled
172039 - Bug 30295: Use blessed patron to enqueue notice and QA fixes
172041 - Bug 30295: Add phone transport option

Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 30295: Send a notification when a recall is manually cancelled
Using index info to reconstruct a base tree...
M	installer/data/mysql/en/mandatory/sample_notices.yml
M	installer/data/mysql/mandatory/sample_notices_message_transports.sql
Falling back to patching base and 3-way merge...
Auto-merging installer/data/mysql/mandatory/sample_notices_message_transports.sql
Auto-merging installer/data/mysql/en/mandatory/sample_notices.yml
CONFLICT (content): Merge conflict in installer/data/mysql/en/mandatory/sample_notices.yml
error: Failed to merge in the changes.
Patch failed at 0001 Bug 30295: Send a notification when a recall is manually cancelled