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

feature/OVPAY-96 #8

Closed
maxmartens wants to merge 0 commits from feature/OVPAY-96 into develop
pull from: feature/OVPAY-96
merge into: HTM:develop
Conversation 0 Commits - Files Changed -
maxmartens commented 2024-12-03 12:47:00 +00:00
Owner
Copy Link
No description provided.
maxmartens added 4 commits 2024-12-03 12:47:00 +00:00 
Added Get Customer V2- first draft
maxmartens changed title from feature/OVPAY-96 to WIP: feature/OVPAY-96 2024-12-03 12:47:06 +00:00
maxmartens requested review from bboterm 2024-12-03 12:48:42 +00:00
maxmartens requested review from MirjamHTM 2024-12-03 12:48:42 +00:00
bboterm requested changes 2024-12-03 13:41:52 +00:00
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +45,4 @@
schema:
type: string
example: Doe
- name: emailAdress
bboterm commented 2024-12-03 13:27:09 +00:00
Owner
Copy Link

Typo: emailAddress (dubbel d, dubbel s)

Typo: emailAddress (dubbel d, dubbel s)
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +97,4 @@
schema:
type: integer
example: 0123456789
- name: OvChipcardAlias
bboterm commented 2024-12-03 13:29:03 +00:00
Owner
Copy Link

Waarom is deze nou ineens met een hoofdletter?

Waarom is deze nou ineens met een hoofdletter?
maxmartens commented 2024-12-03 14:36:31 +00:00
Author
Owner
Copy Link

Omdat Mirjam dat getypt heeft love you Mirjam

~~Omdat Mirjam dat getypt heeft~~ love you Mirjam
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml
@ -0,0 +158,4 @@
"detail": "Meer dan 1 klantprofiel gevonden. Verfijn je zoekcriteria.",
"instance": "555d00b5-bc3f-4591-949b-479e76d49ea7"
}
/customers/tokens:
bboterm commented 2024-12-03 13:35:23 +00:00
Owner
Copy Link

Tobby schrijft voor dat dit endpoint /customers/{customerId}/tokens moet zijn. Doen we dat bewust niet omdat customerId in de JWT zit? Vindt Toby dat OK?

Tobby schrijft voor dat dit endpoint `/customers/{customerId}/tokens` moet zijn. Doen we dat bewust niet omdat customerId in de JWT zit? Vindt Toby dat OK?
maxmartens commented 2024-12-03 14:36:13 +00:00
Author
Owner
Copy Link

@bboterm Dit is nu al zo gebouwd en is voor alle /customers endpoints in SE/Touchpoint nu zo - customerNumber/profileId wordt uit headers gehaald (uit JWT of uit SMP-gevulde header). Voordeel is dat je de customerNumber als touchpoint niet in de URI hoeft mee te geven. Sterker nog, als touchpoint weet je dat niet eens perse - in de klant JWT staat een Azure joejoe ID en in IL wordt die in database opgezocht om customerNumber te verkrijgen.

Dit aanpassen is dus niet triviaal, zeker omdat je dan als touchpoint, om je customerProfileId te verkrijgen, eerst een GET /customers/{customerProfileId} moet doen. Da wordt lastig

@bboterm Dit is nu al zo gebouwd en is voor alle /customers endpoints in SE/Touchpoint nu zo - customerNumber/profileId wordt uit headers gehaald (uit JWT of uit SMP-gevulde header). Voordeel is dat je de customerNumber als touchpoint niet in de URI hoeft mee te geven. Sterker nog, als touchpoint weet je dat niet eens perse - in de klant JWT staat een Azure joejoe ID en in IL wordt die in database opgezocht om customerNumber te verkrijgen. Dit aanpassen is dus niet triviaal, zeker omdat je dan als touchpoint, om je customerProfileId te verkrijgen, eerst een GET /customers/{customerProfileId} moet doen. Da wordt lastig
bboterm marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +171,4 @@
application/json:
schema:
$ref: '#/components/schemas/OvPayTokensResponse'
/customers/tokens/{tokenId}/product-instances:
bboterm commented 2024-12-03 13:37:52 +00:00
Owner
Copy Link

API richtlijnen van Tobby schrijven voor dat je /customers weg kunt laten. Het tokenId is immers al uniek. Tevens is dit pad nu langer dan 3 delen, en dat mag niet.

API richtlijnen van Tobby schrijven voor dat je /customers weg kunt laten. Het tokenId is immers al uniek. Tevens is dit pad nu langer dan 3 delen, en dat mag niet.
bboterm commented 2024-12-03 13:38:21 +00:00
Owner
Copy Link

Daarbij, sinds wanneer doen wij aan kebab case? We doen overal gewoon lowercase in urls.

Daarbij, sinds wanneer doen wij aan kebab case? We doen overal gewoon lowercase in urls.
maxmartens commented 2024-12-03 14:32:17 +00:00
Author
Owner
Copy Link

@bboterm Ik heb /customers ervoor staan omdat:

  • ovpaytokens in het customers datamodel zitten
  • ze aan een customer gelinkt zijn
  • het nu al zo gebouwd is

Ik ben het in principe wel met je eens, dat het in het datamodel onder customers hangt, betekent niet dat het in API URI ook zo moet. /tokens is dan wel erg algemeen zonder context (dan kunnen het ook barrier tokens zijn), dan moeten we er /ovpaytokens van maken.

De URI aanpassen zal dan wel een breaking change zijn voor o.a. Perplex, TENZIJ ze de HATEOAS ook al gebruiken om de URI te bepalen en dan werkt het automagisch :o

@bboterm Ik heb /customers ervoor staan omdat: - ovpaytokens in het customers datamodel zitten - ze aan een customer gelinkt zijn - het nu al zo gebouwd is Ik ben het in principe wel met je eens, dat het in het datamodel onder customers hangt, betekent niet dat het in API URI ook zo moet. /tokens is dan wel erg algemeen zonder context (dan kunnen het ook barrier tokens zijn), dan moeten we er /ovpaytokens van maken. De URI aanpassen zal dan wel een breaking change zijn voor o.a. Perplex, TENZIJ ze de HATEOAS ook al gebruiken om de URI te bepalen en dan werkt het automagisch :o
bboterm marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +243,4 @@
type: string
format: date
example: '2023-02-01'
emailAdress:
bboterm commented 2024-12-03 13:27:42 +00:00
Owner
Copy Link

Typo: emailAddress (dubbel d, dubbel s)

Typo: emailAddress (dubbel d, dubbel s)
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +266,4 @@
example: 1
name:
type: string
example: Brug
bboterm commented 2024-12-03 13:31:23 +00:00
Owner
Copy Link

Voer hier liefst gewoon 'Home' in.

Voer hier liefst gewoon 'Home' in.
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +316,4 @@
example: 1
name:
type: string
example: Mobiel
bboterm commented 2024-12-03 13:32:28 +00:00
Owner
Copy Link

Vul hier ook gewoon 'Home' in.

Vul hier ook gewoon 'Home' in.
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +392,4 @@
{ name: surname, required: false, type: string },
{ name: suffix, required: false, type: string },
{ name: dateOfBirth, required: false, type: string },
{ name: emailAdress, required: false, type: string },
bboterm commented 2024-12-03 13:27:50 +00:00
Owner
Copy Link

Typo: emailAddress (dubbel d, dubbel s)

Typo: emailAddress (dubbel d, dubbel s)
👍 1
maxmartens marked this conversation as resolved
src/openapi/customers/SE-customers.yaml Outdated
@ -0,0 +435,4 @@
required:
- ovPayTokens
properties:
Entries:
bboterm commented 2024-12-03 13:36:14 +00:00
Owner
Copy Link

Waarom schrijf je hier 'Entries' en niet gewoon wat het zijn? 'ovPayTokens' ?

Waarom schrijf je hier 'Entries' en niet gewoon wat het zijn? 'ovPayTokens' ?
maxmartens commented 2024-12-03 14:28:23 +00:00
Author
Owner
Copy Link

Had ik ook al gezien en gefixt, maar nog niet gepusht, snelle jongeman die je bent

Had ik ook al gezien en gefixt, maar nog niet gepusht, snelle jongeman die je bent
maxmartens marked this conversation as resolved
maxmartens added 1 commit 2024-12-03 14:40:53 +00:00
bboterm closed this pull request 2024-12-03 14:44:28 +00:00
maxmartens reopened this pull request 2024-12-04 16:43:12 +00:00
maxmartens changed title from WIP: feature/OVPAY-96 to feature/OVPAY-96 2024-12-04 16:43:28 +00:00
maxmartens requested review from bboterm 2024-12-04 16:46:22 +00:00
maxmartens added 2 commits 2024-12-05 14:28:11 +00:00
maxmartens added 1 commit 2024-12-05 14:48:13 +00:00
bboterm closed this pull request 2025-06-16 08:33:43 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
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
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#8
No description provided.
Delete Branch "feature/OVPAY-96"

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?