Summary: | Add public endpoint for cancelling holds | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Hold requests | Assignee: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Status: | CLOSED FIXED | QA Contact: | Kyle M Hall (khall) <kyle> |
Severity: | normal | ||
Priority: | P5 - low | CC: | andrew, bugzilla, emily.lamancusa, fridolin.somers, gmcharlt, kebliss, kyle, martin.renvoize, rcoert, rkuiper, Slodico, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
23.11.00,23.05.02,22.11.08
|
Circulation function: | |
Attachments: |
Bug 33573: Unit tests
Bug 33573: Add public endpoint for cancelling holds Bug 33573: Unit tests Bug 33573: Add public endpoint for cancelling holds Bug 33573: Unit tests Bug 33573: Add public endpoint for cancelling holds Bug 33573: Unit tests Bug 33573: Add public endpoint for cancelling holds |
Description
Nick Clemens (kidclamp)
2023-04-20 14:22:27 UTC
Created attachment 152107 [details] [review] Bug 33573: Unit tests Created attachment 152108 [details] [review] Bug 33573: Add public endpoint for cancelling holds This patch introduces a public endpoint for cancelling holds. Cancellation requests are generated when the hold is waiting and configuration allows requesting cancellation, as the OPAC does right now. Tests cover all the use cases. To test: 1. Apply this patches 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/patrons_holds.t => SUCCESS: Tests pass! 3. Sign off :-D Created attachment 152112 [details] [review] Bug 33573: Unit tests Signed-off-by: Sam Lau <samalau@gmail.com> Created attachment 152113 [details] [review] Bug 33573: Add public endpoint for cancelling holds This patch introduces a public endpoint for cancelling holds. Cancellation requests are generated when the hold is waiting and configuration allows requesting cancellation, as the OPAC does right now. Tests cover all the use cases. To test: 1. Apply this patches 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/patrons_holds.t => SUCCESS: Tests pass! 3. Sign off :-D Signed-off-by: Sam Lau <samalau@gmail.com> Created attachment 152114 [details] [review] Bug 33573: Unit tests Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Created attachment 152115 [details] [review] Bug 33573: Add public endpoint for cancelling holds This patch introduces a public endpoint for cancelling holds. Cancellation requests are generated when the hold is waiting and configuration allows requesting cancellation, as the OPAC does right now. Tests cover all the use cases. To test: 1. Apply this patches 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/patrons_holds.t => SUCCESS: Tests pass! 3. Sign off :-D Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Our Aspen library is excited to get this function. Created attachment 152904 [details] [review] Bug 33573: Unit tests Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Created attachment 152905 [details] [review] Bug 33573: Add public endpoint for cancelling holds This patch introduces a public endpoint for cancelling holds. Cancellation requests are generated when the hold is waiting and configuration allows requesting cancellation, as the OPAC does right now. Tests cover all the use cases. To test: 1. Apply this patches 2. Run: $ ktd --shell k$ prove t/db_dependent/api/v1/patrons_holds.t => SUCCESS: Tests pass! 3. Sign off :-D Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Pushed to master for 23.11. Nice work everyone, thanks! Any chance for backport on this one? The API endpoint is necessary for the hold cancellation from OPAC feature to work correctly for Koha/Aspen libraries. +1 +1 for the backport to 22.11 and the quick fix. We'd like to be able to turn this on in Aspen, too. We need a decision from the 23.05 RMaint first here. Marking the bug accordingly to get some attention. This shouldn't cause any issues downstream or with end users.. choosing to backport. Thanks for all the hard work! Pushed to 23.05.x for the next release Nice work everyone! Pushed to oldstable for 22.11.x |