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

features/TP-notifications #34

Merged
MirjamHTM merged 16 commits from features/TP-notifications into develop 2025-11-06 09:18:20 +00:00
Conversation 0 Commits 16 Files Changed 3 +1165 -2
MirjamHTM commented 2025-10-30 10:13:37 +00:00
Owner
Copy Link

Endpoints voor SE notifications
Aanpassing CRUD customers rondom isEmailVerified veld
Toevoeging collectie voor Maileon

Endpoints voor SE notifications Aanpassing CRUD customers rondom isEmailVerified veld Toevoeging collectie voor Maileon
MirjamHTM added 11 commits 2025-10-30 10:13:38 +00:00
MirjamHTM requested review from bboterm 2025-10-30 10:13:44 +00:00
MirjamHTM added 1 commit 2025-11-03 13:26:54 +00:00
MirjamHTM added 1 commit 2025-11-03 13:37:27 +00:00
MirjamHTM added 1 commit 2025-11-03 13:47:03 +00:00
bboterm requested changes 2025-11-04 12:05:03 +00:00
src/openapi/customers/SE-notifications.yaml
@ -0,0 +28,4 @@
- name: expand
in: query
schema:
type: array
bboterm commented 2025-11-04 11:58:04 +00:00
Owner
Copy Link

Klopt het dat dit een array is? Kun je meerdere kiezen?

Ik dacht, is een enum niet mooier?

        - name: expand
          in: query
          schema: 
            type: string
            enum: [none, eventType, eventTypeChannel]
Klopt het dat dit een array is? Kun je meerdere kiezen? Ik dacht, is een enum niet mooier? ``` - name: expand in: query schema: type: string enum: [none, eventType, eventTypeChannel] ```
MirjamHTM commented 2025-11-05 13:42:27 +00:00
Author
Owner
Copy Link

Done

Done
src/openapi/customers/SE-notifications.yaml
@ -0,0 +314,4 @@
application/json:
example:
{
"type": "https://api.integratielaag.nl/abt/touchpoint/1.0/customers",
bboterm commented 2025-11-04 11:39:17 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
MirjamHTM marked this conversation as resolved
src/openapi/customers/SE-notifications.yaml
@ -0,0 +328,4 @@
summary: Get all possible notificationSubscriptions for a customer (account or private).
description: |
Get all possible notificationSubscriptions for a customer (account or private), including if they have opted-in for it.
parameters:
bboterm commented 2025-11-04 12:01:04 +00:00
Owner
Copy Link

Klopt het dat alle params niet verplicht zijn?

Klopt het dat alle params niet verplicht zijn?
MirjamHTM commented 2025-11-05 14:03:24 +00:00
Author
Owner
Copy Link

Ja dat klopt, is afhankelijk of een gebruiker ingelogd is via Web of SMP of anoniem opt in op vraagt.

Ja dat klopt, is afhankelijk of een gebruiker ingelogd is via Web of SMP of anoniem opt in op vraagt.
src/openapi/customers/SE-notifications.yaml
@ -0,0 +423,4 @@
content:
application/json:
example:
{
bboterm commented 2025-11-04 12:02:06 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
MirjamHTM marked this conversation as resolved
src/openapi/customers/SE-notifications.yaml Outdated
@ -0,0 +437,4 @@
application/json:
example:
{
"type": "https://api.integratielaag.nl/abt/touchpoint/2.0/notifications",
bboterm commented 2025-11-04 12:02:10 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
MirjamHTM marked this conversation as resolved
src/openapi/customers/SE-notifications.yaml Outdated
@ -0,0 +450,4 @@
summary: Create a new notificationSubscription for a customer (account or private).
description: |
Create a notificationSubscriptions for a customer (account or private), including if they have opted-in for it.
parameters:
bboterm commented 2025-11-04 12:00:35 +00:00
Owner
Copy Link

Klopt het dat alle params niet verplicht zijn?

Klopt het dat alle params niet verplicht zijn?
MirjamHTM commented 2025-11-05 14:05:27 +00:00
Author
Owner
Copy Link

Klopt heeft te maken met hoe het endpoint benaderd wordt, ingelogd, SMP of anoniem.

Klopt heeft te maken met hoe het endpoint benaderd wordt, ingelogd, SMP of anoniem.
src/openapi/customers/SE-notifications.yaml
@ -0,0 +559,4 @@
content:
application/json:
example:
{
bboterm commented 2025-11-04 12:02:23 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
MirjamHTM marked this conversation as resolved
src/openapi/customers/SE-notifications.yaml Outdated
@ -0,0 +574,4 @@
summary: Update a notificationSubscription for a customer (account or private).
description: |
Update a notificationSubscription for a customer (account or private).
parameters:
bboterm commented 2025-11-04 11:59:40 +00:00
Owner
Copy Link

Klopt het dat alle params niet verplicht zijn?

Klopt het dat alle params niet verplicht zijn?
MirjamHTM commented 2025-11-05 14:08:29 +00:00
Author
Owner
Copy Link

Yes zelfde als hier boven

Yes zelfde als hier boven
src/openapi/customers/SE-notifications.yaml Outdated
@ -0,0 +635,4 @@
application/json:
examples:
Update a notificationSubscription to inactive:
summary: Return the updated notification
bboterm commented 2025-11-04 12:03:50 +00:00
Owner
Copy Link

Misschien ff de summary aanpassen. Die is nu gelijk voor beide.

Misschien ff de summary aanpassen. Die is nu gelijk voor beide.
MirjamHTM marked this conversation as resolved
src/openapi/customers/SE-notifications.yaml Outdated
@ -0,0 +646,4 @@
"isActive": False
}
Update a notificationSubscription to active:
summary: Return the updated notification
bboterm commented 2025-11-04 12:03:54 +00:00
Owner
Copy Link

Misschien ff de summary aanpassen. Die is nu gelijk voor beide.

Misschien ff de summary aanpassen. Die is nu gelijk voor beide.
MirjamHTM marked this conversation as resolved
src/openapi/customers/se-maileon.yaml Outdated
@ -0,0 +38,4 @@
required: true
description: The emailadress of the customer
summary: Update a customer profile
description: Update a customer profile based on the customerProfileId
bboterm commented 2025-11-04 11:32:23 +00:00
Owner
Copy Link

... and emailAddress.

... and emailAddress.
MirjamHTM marked this conversation as resolved
src/openapi/customers/se-maileon.yaml
@ -0,0 +54,4 @@
}
responses:
"200":
description: OK
bboterm commented 2025-11-04 11:32:54 +00:00
Owner
Copy Link

Doe je hier bewust geen response body?

Doe je hier bewust geen response body?
MirjamHTM commented 2025-11-05 14:12:09 +00:00
Author
Owner
Copy Link

Ja, ik zat hier nog na te denken, moeten we Maileon alle klant data geven. Ik dacht van niet, dus dacht misschien beter om dit leeg te laten of alleen het Id mee terug te geven, maar wat denk jij?

Ja, ik zat hier nog na te denken, moeten we Maileon alle klant data geven. Ik dacht van niet, dus dacht misschien beter om dit leeg te laten of alleen het Id mee terug te geven, maar wat denk jij?
bboterm commented 2025-11-05 14:37:25 +00:00
Owner
Copy Link

Ik dacht al dat dat de rationale zou zijn. Leeglaten is prima.

Ik dacht al dat dat de rationale zou zijn. Leeglaten is prima.
src/openapi/customers/se-maileon.yaml
@ -0,0 +66,4 @@
- name: expand
in: query
schema:
type: array
bboterm commented 2025-11-04 11:58:58 +00:00
Owner
Copy Link

Klopt het dat dit een array is? Kun je meerdere kiezen?

Ik dacht, is een enum niet mooier?

        - name: expand
          in: query
          schema: 
            type: string
            enum: [none, eventType, eventTypeChannel]
Klopt het dat dit een array is? Kun je meerdere kiezen? Ik dacht, is een enum niet mooier? ``` - name: expand in: query schema: type: string enum: [none, eventType, eventTypeChannel] ```
src/openapi/customers/se-maileon.yaml
@ -0,0 +352,4 @@
application/json:
example:
{
"type": "https://api.integratielaag.nl/abt/touchpoint/1.0/customers",
bboterm commented 2025-11-04 11:34:31 +00:00
Owner
Copy Link

SVP deze structuur even updaten naar: https://htm-prod.atlassian.net/browse/OVPAY-1499

SVP deze structuur even updaten naar: https://htm-prod.atlassian.net/browse/OVPAY-1499
src/openapi/customers/se-maileon.yaml Outdated
@ -0,0 +367,4 @@
description: |
Get all possible notificationSubscriptions for a customer (account or private), including if they have opted-in for it.
parameters:
- name: customerProfileId
bboterm commented 2025-11-04 11:35:22 +00:00
Owner
Copy Link

Klopt het dat deze niet verplicht is?

Klopt het dat deze niet verplicht is?
MirjamHTM commented 2025-11-05 14:25:11 +00:00
Author
Owner
Copy Link

Nope, dat was een fout van mijn kant

Nope, dat was een fout van mijn kant
MirjamHTM marked this conversation as resolved
src/openapi/customers/se-maileon.yaml
@ -0,0 +370,4 @@
- name: customerProfileId
in: header
schema:
type: string
bboterm commented 2025-11-04 11:35:31 +00:00
Owner
Copy Link

CustomerProfileId is een integer.

CustomerProfileId is een integer.
MirjamHTM marked this conversation as resolved
src/openapi/customers/se-maileon.yaml
@ -0,0 +425,4 @@
application/json:
example:
{
"type": "https://api.integratielaag.nl/abt/touchpoint/2.0/notifications",
bboterm commented 2025-11-04 11:36:03 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
src/openapi/customers/se-maileon.yaml Outdated
@ -0,0 +438,4 @@
application/json:
example:
{
"type": "https://api.integratielaag.nl/abt/touchpoint/2.0/notifications",
bboterm commented 2025-11-04 11:36:07 +00:00
Owner
Copy Link

https://htm-prod.atlassian.net/browse/OVPAY-1499

https://htm-prod.atlassian.net/browse/OVPAY-1499
MirjamHTM added 1 commit 2025-11-05 14:28:50 +00:00
MirjamHTM added 1 commit 2025-11-06 09:17:49 +00:00
MirjamHTM merged commit 47204b3b75 into develop 2025-11-06 09:18:20 +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#34
No description provided.
Delete Branch "features/TP-notifications"

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?