Bug 30295 - Send a notification when a recall is manually cancelled
Summary: Send a notification when a recall is manually cancelled
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Aleisha Amohia
QA Contact: Testopia
URL:
Keywords:
Depends on: 23781
Blocks: 29734
  Show dependency treegraph
 
Reported: 2022-03-15 01:18 UTC by Aleisha Amohia
Modified: 2024-02-23 14:49 UTC (History)
8 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 30295: Send a notification when a recall is manually cancelled (15.09 KB, patch)
2022-03-16 02:00 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (13.29 KB, patch)
2022-09-07 04:54 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.36 KB, patch)
2022-09-07 04:58 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.20 KB, patch)
2023-02-22 04:00 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.22 KB, patch)
2023-03-13 00:16 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.26 KB, patch)
2023-04-04 13:33 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.61 KB, patch)
2023-04-30 21:21 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Use blessed patron to enqueue notice and QA fixes (6.85 KB, patch)
2023-09-07 23:15 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Use blessed patron to enqueue notice and QA fixes (6.89 KB, patch)
2023-09-08 06:53 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.41 KB, patch)
2023-11-09 03:09 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Use blessed patron to enqueue notice and QA fixes (6.90 KB, patch)
2023-11-09 03:09 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Add phone transport option (2.26 KB, patch)
2023-11-09 03:09 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 30295: Send a notification when a recall is manually cancelled (12.44 KB, patch)
2023-11-15 20:41 UTC, David Nind
Details | Diff | Splinter Review
Bug 30295: Use blessed patron to enqueue notice and QA fixes (6.94 KB, patch)
2023-11-15 20:41 UTC, David Nind
Details | Diff | Splinter Review
Bug 30295: Add phone transport option (2.30 KB, patch)
2023-11-15 20:41 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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
Created attachment 131763 [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
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
Created attachment 140243 [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
Comment 4 Aleisha Amohia 2022-09-07 04:58:57 UTC
Created attachment 140245 [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
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
Created attachment 147108 [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
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
Created attachment 148114 [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
Comment 9 ByWater Sandboxes 2023-04-04 13:33:26 UTC
Created attachment 149127 [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>
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
Created attachment 150431 [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>
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
Created attachment 155347 [details] [review]
Bug 30295: Use blessed patron to enqueue notice and QA fixes
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
Created attachment 155358 [details] [review]
Bug 30295: Use blessed patron to enqueue notice and QA fixes

Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Comment 16 Stina Hallin 2023-09-08 06:54:44 UTC
Worked as described in test plan
Comment 17 Kyle M Hall 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
Created attachment 158699 [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>
Comment 19 Aleisha Amohia 2023-11-09 03:09:21 UTC
Created attachment 158700 [details] [review]
Bug 30295: Use blessed patron to enqueue notice and QA fixes

Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Comment 20 Aleisha Amohia 2023-11-09 03:09:24 UTC
Created attachment 158701 [details] [review]
Bug 30295: Add phone transport option
Comment 21 David Nind 2023-11-15 20:41:26 UTC
Created attachment 159007 [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 22 David Nind 2023-11-15 20:41:29 UTC
Created attachment 159008 [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 23 David Nind 2023-11-15 20:41:32 UTC
Created attachment 159009 [details] [review]
Bug 30295: Add phone transport option

Signed-off-by: David Nind <david@davidnind.com>
Comment 24 Kyle M Hall 2024-02-23 14:49:24 UTC
This needs to have support for the phone transport as well.