Bug 26281 - Add cancellation reason to holds history
Summary: Add cancellation reason to holds history
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Kyle M Hall (khall)
QA Contact: Testopia
URL:
Keywords:
Depends on: 25534
Blocks:
  Show dependency tree / graph
 
Reported: 2020-08-22 09:46 UTC by Katrin Fischer
Modified: 2021-06-14 21:33 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This adds the new column for cancellation reason to the holds history page in the staff interface. See bug 25534 for more details on the new hold cancellation reason feature.
Version(s) released in:
20.11.00
Circulation function:

Attachments
Bug 26281: Add cancellation reason to holds history (1.43 KB, patch)
2020-08-24 13:25 UTC, Kyle M Hall (khall) 		Details | Diff | Splinter Review
Bug 26281: Add cancellation reason to holds history (1.48 KB, patch)
2020-08-25 13:45 UTC, Amit Gupta 		Details | Diff | Splinter Review
Bug 26281: (QA follow-up) Fix missing filter (1.15 KB, patch)
2020-08-26 21:20 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2020-08-22 09:46:26 UTC 
Bug 25534 adds the ability to store the cancellation reason when a hold is cancelled and communicate it to the user. It would be great if the cancellation reason could also be shown on the holds history.
Comment 1 Kyle M Hall (khall) 2020-08-24 13:25:49 UTC 
Created attachment 108987 [details] [review]
Bug 26281: Add cancellation reason to holds history

Bug 25534 adds the ability to store the cancellation reason when a hold
is cancelled and communicate it to the user. It would be great if the
cancellation reason could also be shown on the holds history.

Test Plan:
1) Apply this patch
2) Cancel a hold with a reason
4) Browse to holds history for the holds' patron
5) Note the reason shows in the status column if one was provided
Comment 2 Amit Gupta 2020-08-25 13:45:02 UTC 
Created attachment 109072 [details] [review]
Bug 26281: Add cancellation reason to holds history

Test Plan:
1) Apply this patch
2) Cancel a hold with a reason
4) Browse to holds history for the holds' patron
5) Note the reason shows in the status column if one was provided
Comment 3 Katrin Fischer 2020-08-26 21:20:26 UTC 
Created attachment 109207 [details] [review]
Bug 26281: (QA follow-up) Fix missing filter

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2020-08-31 14:21:14 UTC 
Pushed to master for 20.11, thanks to everybody involved!
Comment 5 Lucas Gass (lukeg) 2020-09-04 20:34:23 UTC 
enhancement will not be backported to 20.05.x