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 vraag 1.2: minderjarig en niet overleden #65

Merged
merged 25 commits into from
Nov 19, 2024

Conversation

fsamwel
Copy link
Contributor

@fsamwel fsamwel commented Aug 27, 2024

feature voor vraag "1.2 Is persoon a. minderjarig en b. niet overleden?"

Gegeven stappen zijn nog in traditionele manier opgesteld. Dus nog niet volgens #59 of BRP-API/brp-shared#10

@fsamwel fsamwel changed the title Feature minderjarig niet overleden Feature vraag 1.2: minderjarig en niet overleden Aug 27, 2024
| minderjarig | 010000 | hele categorie persoon | geboortedatum van de persoon, geboorteland van persoon |
| minderjarig | 010300 | hele groep geboorte | geboortedatum van de persoon, geboorteland van persoon |
| minderjarig | 010310 | geboortedatum | geboortedatum van de persoon |
| meerderjarig | 010310 | geboortedatum | geboortedatum van de persoon |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Iedereen met een geboortedatum in onderzoek, zelfs als een persoon 50 jaar oud is, krijgt gezag niet te bepalen?
Ik zou verwachten dat als iemand meerderjarig is, geen gezag wordt geretourneerd. Als we dit al willen doen, dan lijkt me dit alleen opportuun voor personen met een leeftijd net boven de 18 oid.

Copy link
Collaborator

@Patrick-4488 Patrick-4488 left a 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 geheel leesbaarder wordt als de features zijn uitgeschreven in aparte scenarios inplaats van in abstracte scenarios.

De test dat een ander gegeven in onderzoek staat dan geboortedatum en er dus wel uitspraak komt zou ik weg laten, dat test meer iets technische dat de code goed moet doen en niet iets wat te maken heeft met minderjarig of niet overleden zijn

PS: ik zou de wijzegingen kunnen maken als je wilt

@FrozenSync
Copy link
Contributor

Requesting approval @fsamwel @CathyDingemanse @Patrick-4488

@FrozenSync FrozenSync merged commit 2bcb7fb into main Nov 19, 2024
2 checks passed
@FrozenSync FrozenSync deleted the feature-minderjarig-niet-overleden branch November 19, 2024 07:37
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