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 +199 -58
MirjamHTM commented 2025-12-15 10:51:18 +00:00
Owner
Copy Link

Changes to add vouchers and deviceId to the CRUD for orders.

Changes to add vouchers and deviceId to the CRUD for orders.
MirjamHTM added 8 commits 2025-12-15 10:51:18 +00:00
MirjamHTM added 1 commit 2025-12-15 14:37:00 +00:00
bboterm requested changes 2025-12-15 16:08:41 +00:00
Dismissed
src/openapi/orders/orders-crud.yaml Outdated
@ -53,0 +60,4 @@
name: deviceId
schema:
type: string
example: "7a28bd54-7ca9-499a-a722-d15ab858ab99"
bboterm commented 2025-12-15 16:00:44 +00:00
Owner
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
Author
Owner
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
Owner
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
src/openapi/orders/orders-crud.yaml
@ -151,0 +174,4 @@
{
"orderVoucherId": "399bd3b3-9721-4f09-a936-d64637de1621",
"issuedVoucher":{
"issuedVoucherId": "a0996218-bc5e-4826-9020-cda98a32838d",
bboterm commented 2025-12-15 16:03:31 +00:00
Owner
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
Author
Owner
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
Owner
Copy Link

Ik zou de status en de claims weglaten idd.

Ik zou de status en de claims weglaten idd.
👍 1
src/openapi/orders/orders-crud.yaml Outdated
@ -718,0 +814,4 @@
example:
[
{
"orderVoucherId": "19ef6882-8eda-43bf-b48e-9b4ff8745a50",
bboterm commented 2025-12-15 16:08:16 +00:00
Owner
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
Author
Owner
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
Owner
Copy Link

Okido, volgens mij kun je het nu fixen.

Okido, volgens mij kun je het nu fixen.
MirjamHTM added 1 commit 2025-12-16 13:53:16 +00:00
bboterm approved these changes 2025-12-16 13:56:28 +00:00
MirjamHTM merged commit 4bd216bd37 into develop 2025-12-16 13:57:40 +00:00
MirjamHTM deleted branch features/OVPAY2294-vouchers-in-orders 2025-12-16 13:57:41 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
bboterm
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: HTM/ovpay#45
No description provided.
Delete Branch "features/OVPAY2294-vouchers-in-orders"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?