From 02e04fba6273121b3fc455d31be250bcb9d1b0d7 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. --- 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" } } -- 1.9.1