HTM/ovpay
4
0
Fork 0
Code Issues Pull Requests 3 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 +10 -24
1 changed files with 10 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit bfaa1ddcb1 - Show all commits

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

@ -60,6 +60,7 @@ paths:
name: deviceId
schema:
type: string
format: uuid
bboterm commented 2025-12-15 16:00:44 +00:00
Outdated
Review
Copy Link

je zou hier nog format: kunnen toevoegen

je zou hier nog `format:` kunnen toevoegen
MirjamHTM commented 2025-12-16 11:20:13 +00:00
Outdated
Review
Copy Link

Waren we al zeker dat deviceId altijd een uuid zou zijn, of was het nog wellicht een string? Als we hier zeker van zijn, voeg ik met alle liefde format toe ;)

Waren we al zeker dat deviceId altijd een uuid zou zijn, of was het nog wellicht een string? Als we hier zeker van zijn, voeg ik met alle liefde format toe ;)
bboterm commented 2025-12-16 12:05:18 +00:00
Outdated
Review
Copy Link

Dit zijn de CRUD's toch? Daar is deviceId gewoon het id van de Device-tabel die we zelf gebouwd hebben.

Wat Sven straks gaat aanleveren wordt waarschijnlijk Device.externalDeviceId. Daar weten we officieel het datatype nog niet van.

Dit zijn de CRUD's toch? Daar is deviceId gewoon het id van de Device-tabel die we zelf gebouwd hebben. Wat Sven straks gaat aanleveren wordt waarschijnlijk Device.externalDeviceId. Daar weten we officieel het datatype nog niet van.
👍 1
example: "7a28bd54-7ca9-499a-a722-d15ab858ab99"
required: false
description: The id of the device used to place the order.
@ -176,18 +177,7 @@ paths:
"issuedVoucher":{
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
"issuedVoucherId": "a0996218-bc5e-4826-9020-cda98a32838d",
"voucherCode": "Voucher1234",
"voucherStatusInstances":
[
{
"voucherStatus":
{
"voucherStatusId": 2,
"name": "issued"
},
"createdOn": "2025-11-22T13:00:00"
}
],
"productId": 31,
"purchasedProductId": 31,
"fromInclusive": "2025-03-22T08:55:00",
"untillInclusive": "2026-03-22T08:55:00"
},
@ -198,18 +188,7 @@ paths:
"issuedVoucher":{
"issuedVoucherId": "54668baf-4905-4e9a-af02-09c170f295ed",
"voucherCode": "Voucher124",
"voucherStatusInstances":
[
{
"voucherStatus":
{
"voucherStatusId": 2,
"name": "issued"
},
"createdOn": "2025-11-22T13:00:00"
}
],
"productId": 35,
"purchasedProductId": 35,
"fromInclusive": "2025-03-22T08:55:00",
"untillInclusive": "2026-03-22T08:55:00"
},
@ -815,6 +794,13 @@ paths:
[
{
"orderVoucherId": "19ef6882-8eda-43bf-b48e-9b4ff8745a50",
"issuedVoucher":{
"issuedVoucherId": "54668baf-4905-4e9a-af02-09c170f295ed",
"voucherCode": "Voucher124",
"purchasedProductId": 35,
"fromInclusive": "2025-03-22T08:55:00",
"untillInclusive": "2026-03-22T08:55:00"
},
"orderId": "f59e4769-53a0-4156-8991-6f9119ba629f",
"orderLineId": "eeb86071-4f59-405d-b2be-7d7a77044bfa"
}