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

Updated OpenAPI spec for voucher supplier, with examples more relevant to the first usecase (ooievaarspas AOW) - this YAML is also shared with PASS devs #60

Merged
maxmartens merged 9 commits from feature/OVPAY-2545-vouchers into develop 2026-06-08 08:45:46 +00:00
Conversation 1 Commits 9 Files Changed 2 +1032 -271
maxmartens commented 2026-05-07 07:53:44 +00:00
Owner
Copy Link
No description provided.
MirjamHTM was assigned by maxmartens 2026-05-07 07:53:44 +00:00
bboterm was assigned by maxmartens 2026-05-07 07:53:44 +00:00
ovpay was assigned by maxmartens 2026-05-07 07:53:44 +00:00
maxmartens added 3 commits 2026-05-07 07:53:44 +00:00
maxmartens requested review from ovpay 2026-05-07 07:53:52 +00:00
maxmartens requested review from bboterm 2026-05-07 07:53:52 +00:00
maxmartens requested review from MirjamHTM 2026-05-07 07:53:52 +00:00
bboterm requested changes 2026-05-20 09:59:10 +00:00
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -332,4 +307,3 @@
},
],
}
Voucher for a whole order:
bboterm commented 2026-05-20 09:55:43 +00:00
Owner
Copy Link

Is dit niet meer van toepassing?

Is dit niet meer van toepassing?
maxmartens commented 2026-05-20 11:18:10 +00:00
Author
Owner
Copy Link

Deze moest ik er van Paul uithalen voor PASS, en sowieso is complete winkelmand korting iets wat we nog niet echt besproken hadden toen ik dit maakte en wat ook niet formeel in scope is; de MVP is nu vooral bedoeld voor ooievaarspas.
Ik vind zeker prima om examples voor winkelmandkorting + kortingsbedrag op een specifiek product weer erin te hangen, maar voor nu is iig belangrijkst dat we de ooievaarspas case helemaal scherp hebben, want die wordt het eerst gebruikt

Deze moest ik er van Paul uithalen voor PASS, en sowieso is complete winkelmand korting iets wat we nog niet echt besproken hadden toen ik dit maakte en wat ook niet formeel in scope is; de MVP is nu vooral bedoeld voor ooievaarspas. Ik vind zeker prima om examples voor winkelmandkorting + kortingsbedrag op een specifiek product weer erin te hangen, maar voor nu is iig belangrijkst dat we de ooievaarspas case helemaal scherp hebben, want die wordt het eerst gebruikt
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -321,4 +304,1 @@
"value": "1980-06-31",
},
{
"mandatoryCustomerDataItem":
bboterm commented 2026-05-20 09:48:48 +00:00
Owner
Copy Link

Waarom halen we deze eruit?

Waarom halen we deze eruit?
maxmartens commented 2026-05-20 11:15:41 +00:00
Author
Owner
Copy Link

Voor ooievaarspas-product gaat PASS alleen PAD geboortedatum als claim meegeven, dus ooievaarspashouders kunnen straks met elk willekeurig emailadres inwisselen

Voor ooievaarspas-product gaat PASS alleen PAD geboortedatum als claim meegeven, dus ooievaarspashouders kunnen straks met elk willekeurig emailadres inwisselen
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -291,4 +290,1 @@
"productId": 126,
"productName": "HTM-30001",
"productDescription": "Regiovrij maand.",
"_links":
bboterm commented 2026-05-20 09:42:50 +00:00
Owner
Copy Link

Halen we die HATEOAS er bewust uit? Kan me voorstellen dat supplier niet per definitie dit endpoint aan kan roepen, dus dat het niet zoveel nut heeft.

Halen we die HATEOAS er bewust uit? Kan me voorstellen dat supplier niet per definitie dit endpoint aan kan roepen, dus dat het niet zoveel nut heeft.
maxmartens commented 2026-06-01 06:17:55 +00:00
Author
Owner
Copy Link

Dit idd, in de TP versie lijkt het me wel nuttig, voor de supplier niet relevant (of zelfs niet toegestaan om te zien idd)

Dit idd, in de TP versie lijkt het me wel nuttig, voor de supplier niet relevant (of zelfs niet toegestaan om te zien idd)
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml
@ -15,3 +15,1 @@
Get a list of all voucher definitions that the calling touch point is allowed to issue.
Essentially, this means that only products that have active sellingPeriods for touch points within the same
retailer as the calling touch point are returned.
Get a list of all voucher definitions that the calling touchpoint is allowed to issue.\
bboterm commented 2026-05-20 07:57:13 +00:00
Owner
Copy Link

WAAROM DIE BACKSLASH.

WAAROM DIE BACKSLASH.
maxmartens commented 2026-06-01 06:15:02 +00:00
Author
Owner
Copy Link

Is dit mooie-formatting shaming? :O

Is dit mooie-formatting shaming? :O
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -29,3 +36,3 @@
in: query
required: false
description: Filter the voucher definitions on a specific product id.
description: Return a specific voucher definition by it's productId.
bboterm commented 2026-05-20 09:30:10 +00:00
Owner
Copy Link

its

its
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -231,3 +234,3 @@
in: query
required: false
description: The unique identifier of the product for which to retrieve all issued vouchers.
description: Return only issued vouchers that were issued for the given voucher definition (by it's productId).
bboterm commented 2026-05-20 09:37:03 +00:00
Owner
Copy Link

*its

*its
❤️ 1
maxmartens marked this conversation as resolved
src/openapi/products/SE-vouchers-supplier.yaml Outdated
@ -437,1 +371,3 @@
summary: Issue a voucher with prefilled voucher code
Issue a voucher and supply own voucher code:
summary: Issue a voucher and supply own voucher code
description: This allows the voucher supplier to supply its own voucher code, which can be useful if the suplier already has its own (internal or external) source for voucher codes. The supplied voucher code must be unique - if this code is already in use, the request will fail.
bboterm commented 2026-05-20 09:32:40 +00:00
Owner
Copy Link

*supplier

*supplier
maxmartens commented 2026-05-20 11:18:34 +00:00
Author
Owner
Copy Link

i no use AI, dis is proof

i no use AI, dis is proof
maxmartens marked this conversation as resolved
maxmartens added 1 commit 2026-06-01 08:40:42 +00:00
maxmartens added 1 commit 2026-06-01 09:17:28 +00:00
maxmartens added 1 commit 2026-06-02 15:21:06 +00:00 
Fix some typos

Signed-off-by: Max Martens <m.martens@htm.nl>
maxmartens added 2 commits 2026-06-03 17:06:48 +00:00
MirjamHTM commented 2026-06-05 14:28:29 +00:00
Owner
Copy Link

@maxmartens Voor consistentie misschien in de GET voucherdefinitions, de ooievaarspas prijs op 0 zetten, weerspiegelt de werkelijkheid ;) Voor de rest ziet de supplier kant er goed uit!

@maxmartens Voor consistentie misschien in de GET voucherdefinitions, de ooievaarspas prijs op 0 zetten, weerspiegelt de werkelijkheid ;) Voor de rest ziet de supplier kant er goed uit!
maxmartens added 1 commit 2026-06-08 08:44:19 +00:00
maxmartens merged commit e5a0b52d1b into develop 2026-06-08 08:45:46 +00:00
maxmartens deleted branch feature/OVPAY-2545-vouchers 2026-06-08 08:45:47 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
ovpay
MirjamHTM
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
MirjamHTM
bboterm
ovpay
3 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#60
No description provided.
Delete Branch "feature/OVPAY-2545-vouchers"

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?