Skip to content
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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

fsamwel
Copy link
Contributor

@fsamwel fsamwel commented Aug 30, 2024

N.B. step automation moet nog worden toegevoegd. Stappen zijn gemaakt conform BRP-API/brp-shared#10

* 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'
Copy link
Contributor

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?

Copy link
Collaborator

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

@Patrick-4488
Copy link
Collaborator

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 allemaal kinderen uit de testset van de BRP dat maakt het voor een grotere doelgroep mogelijk herkenbaar.

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

Copy link
Contributor

@CathyDingemanse CathyDingemanse left a 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants