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").
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
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>
rebased on bug 16699
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.
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>
Pushed to master for 16.11, thanks Lari!
Blocked by deps, skipping for 16.05.x