From ac3164b5f38c08adf3df114389eb0af2700c519e Mon Sep 17 00:00:00 2001 From: Lari Taskula Date: Thu, 11 Aug 2016 15:51:38 +0300 Subject: [PATCH] Bug 16699: Remove requirement from borrowernumberQueryParam borrowernumberQueryParam shouldn't be required as also changed in Bug 16271. To test: 1. Don't apply the patch yet, but first minify Swagger and run t/db_dependent/api/v1/holds.t 2. Observe that some tests fail with response code 400 when expecting 200. 3. Apply patch and minify Swagger 4. Run t/db_dependent/api/v1/holds.t 5. Observe that tests pass. Signed-off-by: Benjamin Rokseth --- api/v1/swagger/parameters/patron.json | 1 - 1 file changed, 1 deletion(-) diff --git a/api/v1/swagger/parameters/patron.json b/api/v1/swagger/parameters/patron.json index 2b204c1..409d0b2 100644 --- a/api/v1/swagger/parameters/patron.json +++ b/api/v1/swagger/parameters/patron.json @@ -10,7 +10,6 @@ "name": "borrowernumber", "in": "query", "description": "Internal borrower identifier", - "required": true, "type": "integer" } } -- 2.1.4