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

feature/OVPAY-2018 #29

Closed
bboterm wants to merge 0 commits from feature/OVPAY-2018 into develop
pull from: feature/OVPAY-2018
merge into: HTM:develop
Conversation 2 Commits - Files Changed -
bboterm commented 2025-09-09 09:18:33 +00:00
Owner
Copy Link
No description provided.
MirjamHTM was assigned by bboterm 2025-09-09 09:18:33 +00:00
bboterm added 3 commits 2025-09-09 09:18:34 +00:00
bboterm added 1 commit 2025-09-09 12:41:32 +00:00
MirjamHTM commented 2025-10-03 11:45:59 +00:00
Owner
Copy Link

In de SE GET en POST op customer zie ik in het device de pushToken en isPreferred staan, isPreferred lijkt mij niet nodig, staat iig niet in het datamodel, voor de pushToken slaan we ook niet op in de tabel dus die zouden we op moeten halen bij de notificatie service. Willen we dat doen, wat moet het touchpoint doen met het pushToken? In beide calls staat het veld alias btw niet voor de device. Er wordt ook het deviceType mee gegeven bij de GET, die moet dan ook ergens anders vandaan komen of niet? Voor de rest ziet het er goed uit, moet je alleen nog een mergeconflict oplossen :)

In de SE GET en POST op customer zie ik in het device de pushToken en isPreferred staan, isPreferred lijkt mij niet nodig, staat iig niet in het datamodel, voor de pushToken slaan we ook niet op in de tabel dus die zouden we op moeten halen bij de notificatie service. Willen we dat doen, wat moet het touchpoint doen met het pushToken? In beide calls staat het veld alias btw niet voor de device. Er wordt ook het deviceType mee gegeven bij de GET, die moet dan ook ergens anders vandaan komen of niet? Voor de rest ziet het er goed uit, moet je alleen nog een mergeconflict oplossen :)
bboterm added 1 commit 2025-10-03 12:02:25 +00:00
bboterm commented 2025-10-03 12:03:01 +00:00
Author
Owner
Copy Link

Ah kak, dat was wel heel lelijk. Ook had ik in de SE-variant deviceId een integer gemaakt, maar dat is een UUID. PushToken, wtf waar komt dat vandaan?

Ik heb het nu gefixt. En ja, mergen gaat nog leuk worden. :x

Ah kak, dat was wel heel lelijk. Ook had ik in de SE-variant deviceId een integer gemaakt, maar dat is een UUID. PushToken, wtf waar komt dat vandaan? Ik heb het nu gefixt. En ja, mergen gaat nog leuk worden. :x
bboterm added 1 commit 2025-10-03 13:02:42 +00:00
bboterm closed this pull request 2025-10-03 13:13:22 +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
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
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#29
No description provided.
Delete Branch "feature/OVPAY-2018"

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?