Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

isNevenadres is niet optioneel #301

Open
melsk-r opened this issue Jul 23, 2024 · 1 comment
Open

isNevenadres is niet optioneel #301

melsk-r opened this issue Jul 23, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@melsk-r
Copy link
Collaborator

melsk-r commented Jul 23, 2024

Originally created by KayodeBakker (lvbag/BAG-Gemeentelijke-wensen-tav-BAG-Bevragingen#507):

Het viel op dat bij AdresUitgebreid in het response geen isNevenadres meekomt als het om een hoofdadres gaat, zoals bij 1883200000036667. Ik vroeg me af waarom er voor is gekozen om bij een false geen waarde te tonen. Is het juist niet de bedoeling om hier eenduidig over te zijn? Daarmee dus altijd true of false te tonen.

@melsk-r melsk-r added the enhancement New feature or request label Jul 23, 2024
@melsk-r
Copy link
Collaborator Author

melsk-r commented Jul 23, 2024

This comment originally might have been created by someone else.

Het gaat verder dan alleen één property. Als je dit wilt, dan zou je het ook moeten doen voor de properties van AdresMogelijkOnjuist. Dat is volgens mij geen goed idee.
Ik denk dat het handiger is om te controleren of een adres niet onjuist is door alleen te kijken of de mogelijkOnjuist property null is in plaats van alle AdresMogelijkOnjuist properties langslopen om dan te concluderen dat het adres niet onjuist is.

Verder zijn wij er van uit gegaan dat situaties als isNevenadres, geconstateerd, mogelijkOnjuist veel minder vaak voorkomt. Als je dan false meegeeft voor deze properties, dan zou je de payload alleen maar vullen met veel niet relevante data.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant