Bug 33485 - Add/remove title from holdings is not using the correct endpoint
Summary: Add/remove title from holdings is not using the correct endpoint
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: ERM (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low major (vote)
Assignee: Jonathan Druart
QA Contact: Marcel de Rooy
URL:
Keywords: rel_22_11_candidate
Depends on: 32939
Blocks:
  Show dependency treegraph
 
Reported: 2023-04-11 12:59 UTC by Jonathan Druart
Modified: 2023-12-28 20:43 UTC (History)
6 users (show)

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


Attachments
Bug 33485: (bug 32939 follow-up) Fix title endpoint (1.07 KB, patch)
2023-04-11 13:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 33485: (bug 32939 follow-up) Fix title endpoint (1.12 KB, patch)
2023-04-11 17:15 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 33485: (bug 32939 follow-up) Fix title endpoint (1.22 KB, patch)
2023-04-14 06:27 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2023-04-11 12:59:14 UTC
Bad copy/paste from 32939
Comment 1 Jonathan Druart 2023-04-11 13:00:30 UTC
Created attachment 149442 [details] [review]
Bug 33485: (bug 32939 follow-up) Fix title endpoint

The change is obvious, we shoulf call the title endpoint, not package.
Comment 2 Pedro Amorim 2023-04-11 17:15:20 UTC
Created attachment 149454 [details] [review]
Bug 33485: (bug 32939 follow-up) Fix title endpoint

The change is obvious, we shoulf call the title endpoint, not package.

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 3 Marcel de Rooy 2023-04-14 06:27:35 UTC
Created attachment 149638 [details] [review]
Bug 33485: (bug 32939 follow-up) Fix title endpoint

The change is obvious, we shoulf call the title endpoint, not package.

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Tomás Cohen Arazi 2023-04-14 14:20:29 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 5 Matt Blenkinsop 2023-04-18 12:06:23 UTC
Nice work everyone!

Pushed to stable for 22.11.x