HTM/ovpay
4
0
Fork 0
Code Issues Pull Requests 4 Actions Packages Projects Releases 1 Wiki Activity

features/OVPAY2294-vouchers-in-orders #45

Merged
MirjamHTM merged 10 commits from features/OVPAY2294-vouchers-in-orders into develop 2025-12-16 13:57:40 +00:00
Conversation 0 Commits 10 Files Changed 1 +11 -11
1 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 55bac27561 - Show all commits

22
src/openapi/orders/orders-crud.yaml
View File

@ -170,9 +170,9 @@ paths:
"description": "Betaling in behandeling", "description": "Betaling in behandeling",
}, },
], ],
"order_OrderVouchers": [ "orderVouchers": [
{ {
"order_OrderVoucherId": "399bd3b3-9721-4f09-a936-d64637de1621", "orderVoucherId": "399bd3b3-9721-4f09-a936-d64637de1621",
"issuedVoucher":{ "issuedVoucher":{
"issuedVoucherId": "a0996218-bc5e-4826-9020-cda98a32838d", "issuedVoucherId": "a0996218-bc5e-4826-9020-cda98a32838d",
bboterm commented 2025-12-15 16:03:31 +00:00
Review
Copy Link

Even een checkvraag: kan DHIL zonder moeite tabellen uit een ander datamodel joinen in een CRUD?

Even een checkvraag: kan DHIL zonder moeite tabellen uit een ander datamodel joinen in een CRUD?
MirjamHTM commented 2025-12-16 11:15:27 +00:00
Review
Copy Link

Nou dat doen we natuurlijk al voor touchpoint onder andere, dus ik dacht misschien handig om dat hier gelijk te doen, maar ik zat te twijfelen of ik ook de status en claims moet meegeven, misschien alleen het issuedVoucher object.

Nou dat doen we natuurlijk al voor touchpoint onder andere, dus ik dacht misschien handig om dat hier gelijk te doen, maar ik zat te twijfelen of ik ook de status en claims moet meegeven, misschien alleen het issuedVoucher object.
bboterm commented 2025-12-16 12:05:47 +00:00
Review
Copy Link

Ik zou de status en de claims weglaten idd.

Ik zou de status en de claims weglaten idd.
👍 1
"voucherCode": "Voucher1234", "voucherCode": "Voucher1234",
@ -194,7 +194,7 @@ paths:
"orderLineId": null "orderLineId": null
}, },
{ {
"order_OrderVoucherId": "f6c7ac42-1811-4e4d-82af-53e18fe16110", "orderVoucherId": "f6c7ac42-1811-4e4d-82af-53e18fe16110",
"issuedVoucher":{ "issuedVoucher":{
"issuedVoucherId": "54668baf-4905-4e9a-af02-09c170f295ed", "issuedVoucherId": "54668baf-4905-4e9a-af02-09c170f295ed",
"voucherCode": "Voucher124", "voucherCode": "Voucher124",
@ -406,7 +406,7 @@ paths:
"description": "Concept order", "description": "Concept order",
}, },
], ],
"order_OrderVouchers": "orderVouchers":
[ [
{ {
"issuedVoucherId": "e81b2197-a6c2-45b6-9560-8ce8442e8604", "issuedVoucherId": "e81b2197-a6c2-45b6-9560-8ce8442e8604",
@ -533,7 +533,7 @@ paths:
"description": "Betaling in behandeling", "description": "Betaling in behandeling",
}, },
], ],
"order_OrderVouchers": null, "orderVouchers": null,
"orderLines": "orderLines":
[ [
{ {
@ -773,12 +773,12 @@ paths:
$ref: "#/components/schemas/unavailable" $ref: "#/components/schemas/unavailable"
example: example:
{ {
"order_orderVoucherId": "b9cf0096-4211-4be6-ac21-7bc34bc8e066", "orderVoucherId": "b9cf0096-4211-4be6-ac21-7bc34bc8e066",
} }
/ordervouchers: /ordervouchers:
parameters: parameters:
- in: query - in: query
name: order_OrderVoucherId name: orderVoucherId
schema: schema:
type: string type: string
format: uuid format: uuid
@ -814,15 +814,15 @@ paths:
example: example:
[ [
{ {
"order_OrderVoucherId": "19ef6882-8eda-43bf-b48e-9b4ff8745a50", "orderVoucherId": "19ef6882-8eda-43bf-b48e-9b4ff8745a50",
bboterm commented 2025-12-15 16:08:16 +00:00
Outdated
Review
Copy Link

Waarom inline je de issuedVoucher hier dan weer niet?

Waarom inline je de issuedVoucher hier dan weer niet?
MirjamHTM commented 2025-12-16 11:21:57 +00:00
Outdated
Review
Copy Link

Klopt zou consistent moeten zijn, ik dacht dat ik ze voor allemaal had toegevoegd, iets met find all en replace die niet is goed gegaan!

Klopt zou consistent moeten zijn, ik dacht dat ik ze voor allemaal had toegevoegd, iets met find all en replace die niet is goed gegaan!
bboterm commented 2025-12-16 12:06:04 +00:00
Outdated
Review
Copy Link

Okido, volgens mij kun je het nu fixen.

Okido, volgens mij kun je het nu fixen.
"orderId": "f59e4769-53a0-4156-8991-6f9119ba629f", "orderId": "f59e4769-53a0-4156-8991-6f9119ba629f",
"orderLineId": "eeb86071-4f59-405d-b2be-7d7a77044bfa" "orderLineId": "eeb86071-4f59-405d-b2be-7d7a77044bfa"
} }
] ]
/ordervouchers/{order_ordervoucherId}: /ordervouchers/{ordervoucherId}:
parameters: parameters:
- in: path - in: path
name: order_ordervoucherId name: ordervoucherId
schema: schema:
type: string type: string
format: uuid format: uuid
@ -853,7 +853,7 @@ paths:
$ref: "#/components/schemas/unavailable" $ref: "#/components/schemas/unavailable"
example: example:
{ {
"order_orderVoucherId": "b9cf0096-4211-4be6-ac21-7bc34bc8e066", "orderVoucherId": "b9cf0096-4211-4be6-ac21-7bc34bc8e066",
} }
delete: delete:
summary: Delete an order voucher. summary: Delete an order voucher.