Bug 16699: Add borrowernumberQueryParam for reusability
The borrowernumber as a query parameter should be defined in parameters.json in order to allow its reusability. To test: 1. Apply patch 2. Run minifySwagger.pl 3. Validate swagger.min.json in online.swagger.io/validator/debug?url=url_to+ _your_swagger_min_json or your local swagger-api/validator-badge validator 4. Observe that validation passes Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> My name is Olli-Antti Kivilahti and I approve this commit. We have been using the Swagger2.0-driven REST API on Mojolicious for 1 year now in production and I am certain we have a pretty good idea on how to work with the limitations of Swagger2.0 We participated in the development of the Mojolicious::Plugin::Swagger and know it well. We have made an extension to the plugin to provide full CORS support and have been building all our in-house features on the new REST API. Signed-off-by: Johanna Raisa <johanna.raisa@gmail.com> My name is Johanna Räisä and I approve this commit. We have been using Swagger2.0-driven REST API in production successfully. Signed-off-by: Benjamin Rokseth <benjamin.rokseth@kul.oslo.kommune.no> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This commit is contained in:
parent
0eaf72ab26
commit
baaeb725d2
3 changed files with 11 additions and 4 deletions
|
@ -2,6 +2,9 @@
|
|||
"borrowernumberPathParam": {
|
||||
"$ref": "parameters/patron.json#/borrowernumberPathParam"
|
||||
},
|
||||
"borrowernumberQueryParam": {
|
||||
"$ref": "parameters/patron.json#/borrowernumberQueryParam"
|
||||
},
|
||||
"holdIdPathParam": {
|
||||
"$ref": "parameters/hold.json#/holdIdPathParam"
|
||||
}
|
||||
|
|
|
@ -5,5 +5,12 @@
|
|||
"description": "Internal patron identifier",
|
||||
"required": true,
|
||||
"type": "integer"
|
||||
},
|
||||
"borrowernumberQueryParam": {
|
||||
"name": "borrowernumber",
|
||||
"in": "query",
|
||||
"description": "Internal borrower identifier",
|
||||
"required": true,
|
||||
"type": "integer"
|
||||
}
|
||||
}
|
||||
|
|
|
@ -11,10 +11,7 @@
|
|||
"type": "integer"
|
||||
},
|
||||
{
|
||||
"name": "borrowernumber",
|
||||
"in": "query",
|
||||
"description": "Internal borrower identifier",
|
||||
"type": "integer"
|
||||
"$ref": "../parameters.json#/borrowernumberQueryParam"
|
||||
},
|
||||
{
|
||||
"name": "reservedate",
|
||||
|
|
Loading…
Reference in a new issue