Browse Source

Bug 28272: (QA follow-up) Add 'manager' to cashups definition

The embeddable 'manager' relation field was missing from the
specification file and as such calls using this embed would result in a
500 error since we introduced 'additionalProperties: false'.

Test plan
1/ Enable 'UseCashRegisters'
2/ Navigate to the register details page
3/ Inspect the network traffic and note a 500 error on the cashups api
call
4/ Apply the patch
5/ The 500 should have gone away and been replaced with an empty
resultset 200 page.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
21.05.x
Martin Renvoize 8 months ago
committed by Jonathan Druart
parent
commit
300210e8c3
  1. 7
      api/v1/swagger/definitions/cashup.json

7
api/v1/swagger/definitions/cashup.json

@ -13,6 +13,13 @@
"type": "integer",
"description": "Internal identifier for the manager the cashup was performed by"
},
"manager": {
"type": [
"object",
"null"
],
"description": "The object representing the manager the cashup was performed by"
},
"amount": {
"type": "number",
"description": "Account line amount"

Loading…
Cancel
Save