-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature meerdere burgerservicenummers in één request #77
base: main
Are you sure you want to change the base?
Conversation
features/meerdere-bsns.feature
Outdated
* heeft persoon 'P1' als ouder1 vanaf de geboorteaangifte | ||
* heeft persoon 'P2' als ouder2 vanaf de geboorteaangifte | ||
* is ingeschreven in de de BRP | ||
En persoon 'P1' heeft een kind 'P3' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wordt met deze stap gezegd dat het ook mogelijk is dat bij een persoon is opgenomen dat hij een ouder1/ouder2 heeft, maar bij de betreffende ouder(s) de persoon niet als kind is opgenomen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ik denk dat het in dit geval beter is om in alle gevallen te redeneren vanuit het kind. Want dat is ook wat de gezagsmodule doet.
Dus redeneren vanuit: En persoon 'Kind' heeft als ouder 'Ouder'
Goed om consistent te zijn denk ik
Goede test denk ik om te doen. Mijn voorstel zou zijn in dit geval om nog een stukje achtergrond en scenario schets te geven van wat er getest gaat worden en wat het doel is van deze testen. Ook denk ik dat het mogelijk waardevol kan zijn om een groot gezin toe te voegen zoals het gezin Verder mis ik een tweede gezin, het is mogelijk waarde vol om ook te testen dat als meerdere BSN's opgevraagd worden dat die BSNs inderdaad onderling geen verband hoeven te hebben. PS: ik doe graag een aanpassing als je wil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inderdaad een test om te valideren of het bevragen van meer BSNs ook voor gezag werkt.
Quality Gate passedIssues Measures |
N.B. step automation moet nog worden toegevoegd. Stappen zijn gemaakt conform BRP-API/brp-shared#10