Bug 17086

Summary: REST API: Reword borrowers to patrons in Swagger tags for holds
Product: Koha Reporter: Lari Taskula <lari.taskula>
Component: Web servicesAssignee: Lari Taskula <lari.taskula>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: josef.moravec, kyle, mtj
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 13903    
Bug Blocks:    
Attachments: Bug 17086: Reword borrowers to patrons in Swagger tags for holds
[SIGNED-OFF] Bug 17086: Reword borrowers to patrons in Swagger tags for holds
[PASSED QA] Bug 17086: Reword borrowers to patrons in Swagger tags for holds

Description Lari Taskula 2016-08-09 11:41:18 UTC
Some routes for holds have tags called "borrowers". We should use "patrons" instead in order not to have both (endpoints for patrons already add "patrons").
Comment 1 Lari Taskula 2016-08-09 11:44:42 UTC
Created attachment 54197 [details] [review]
Bug 17086: Reword borrowers to patrons in Swagger tags for holds

Routes for holds have tags called "borrowers". We should use "patrons" instead
in order not to have both (endpoints for patrons already add "patrons").

This patch changes the tags from borrowers to patrons in:
GET  /holds
POST /holds
Comment 2 Josef Moravec 2016-11-08 21:01:26 UTC
Created attachment 57351 [details] [review]
[SIGNED-OFF] Bug 17086: Reword borrowers to patrons in Swagger tags for holds

Routes for holds have tags called "borrowers". We should use "patrons" instead
in order not to have both (endpoints for patrons already add "patrons").

This patch changes the tags from borrowers to patrons in:
GET  /holds
POST /holds

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 3 Josef Moravec 2016-11-08 21:02:16 UTC
rebased on bug 16699
Comment 4 Katrin Fischer 2016-11-20 19:44:48 UTC
I see more issues with terminology in the holds file:

- reserve should be renamed to be hold in the descriptions, example: Internal reserve identifier
- Found status = Hold status (could also be cancelled)
- Branch code = Library code
- Borrower internal identifier = Patron internal identifier

etc.
Comment 5 Katrin Fischer 2016-11-20 19:46:40 UTC
Created attachment 57671 [details] [review]
[PASSED QA] Bug 17086: Reword borrowers to patrons in Swagger tags for holds

Routes for holds have tags called "borrowers". We should use "patrons" instead
in order not to have both (endpoints for patrons already add "patrons").

This patch changes the tags from borrowers to patrons in:
GET  /holds
POST /holds

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Katrin Fischer  <katrin.fischer@bsz-bw.de>
Comment 6 Kyle M Hall (khall) 2016-11-22 11:40:28 UTC
Pushed to master for 16.11, thanks Lari!
Comment 7 Mason James 2016-12-14 11:36:27 UTC
Blocked by deps, skipping for 16.05.x