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

OVPAY-2201 - Added endpoints for contract changes. #54

Merged
bboterm merged 2 commits from feature/OVPAY-2201 into develop 2026-03-04 09:44:38 +00:00
Conversation 2 Commits 2 Files Changed 1 +264 -3
bboterm commented 2026-02-23 13:44:04 +00:00
Owner
Copy Link
No description provided.
MirjamHTM was assigned by bboterm 2026-02-23 13:44:04 +00:00
maxmartens was assigned by bboterm 2026-02-23 13:44:04 +00:00
bboterm added 1 commit 2026-02-23 13:44:05 +00:00
bboterm added 1 commit 2026-03-02 09:05:30 +00:00
MirjamHTM commented 2026-03-04 09:42:19 +00:00
Owner
Copy Link

Ik zie geen rare dingen, enige gedachte is als ik kijk naar de cancellationMoments, dat we ze bijna hadden kunnen samenvoegen wat betreft request en response model is het bijna exact hetzelfde, maar dat zou een breaking change zijn denk ik. En nog even een check vraag vanuit mij : we hebben bewust geen undoChange endpoint toch, als je een change wilt terugdraaien is dat gewoon een nieuwe POST change toch?

Ik zie geen rare dingen, enige gedachte is als ik kijk naar de cancellationMoments, dat we ze bijna hadden kunnen samenvoegen wat betreft request en response model is het bijna exact hetzelfde, maar dat zou een breaking change zijn denk ik. En nog even een check vraag vanuit mij : we hebben bewust geen undoChange endpoint toch, als je een change wilt terugdraaien is dat gewoon een nieuwe POST change toch?
bboterm commented 2026-03-04 09:44:26 +00:00
Author
Owner
Copy Link

Ik zie geen rare dingen, enige gedachte is als ik kijk naar de cancellationMoments, dat we ze bijna hadden kunnen samenvoegen wat betreft request en response model is het bijna exact hetzelfde, maar dat zou een breaking change zijn denk ik. En nog even een check vraag vanuit mij : we hebben bewust geen undoChange endpoint toch, als je een change wilt terugdraaien is dat gewoon een nieuwe POST change toch?

Volgens mij hebben we wel iets gezegd van dat je dan de contractVersion weer weg zou kunnen gooien, maar dat is niet in scope van dit ticket. Je mag overigens niet een change op een change doet als deze klaarstaat in de toekomst.

Ik gaat m mergen.

> Ik zie geen rare dingen, enige gedachte is als ik kijk naar de cancellationMoments, dat we ze bijna hadden kunnen samenvoegen wat betreft request en response model is het bijna exact hetzelfde, maar dat zou een breaking change zijn denk ik. En nog even een check vraag vanuit mij : we hebben bewust geen undoChange endpoint toch, als je een change wilt terugdraaien is dat gewoon een nieuwe POST change toch? Volgens mij hebben we wel iets gezegd van dat je dan de contractVersion weer weg zou kunnen gooien, maar dat is niet in scope van dit ticket. Je mag overigens niet een change op een change doet als deze klaarstaat in de toekomst. Ik gaat m mergen.
bboterm merged commit 2cb6f910df into develop 2026-03-04 09:44:38 +00:00
bboterm deleted branch feature/OVPAY-2201 2026-03-04 09:44:38 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
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
maxmartens
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#54
No description provided.
Delete Branch "feature/OVPAY-2201"

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?