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

opvragen link zonder _links geeft fout #265

Closed
fsamwel opened this issue Jun 19, 2020 · 5 comments
Closed

opvragen link zonder _links geeft fout #265

fsamwel opened this issue Jun 19, 2020 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@fsamwel
Copy link
Collaborator

fsamwel commented Jun 19, 2020

Wanneer in de fields parameter de naam van een _links property wordt opgenomen geeft dit een foutmelding.
Bijvoorbeeld /openbareruimten/0599300000002397?fields=status%2Cwoonplaats geeft een 500 fout.
/openbareruimten/0599300000002397?fields=status%2C_links.woonplaats geeft geen fout, maar geeft het correcte gefilterde antwoord terug

De fields feature zegt hierover: "In de fields parameter kan de relatie worden gevraagd door de relatienaam op te nemen al dan niet voorafgegaan door HAL-element _links."

Dus _links.woonplaats moet te vragen zijn met fields=woonplaats én met fields=_links.woonplaats

@fsamwel fsamwel added the bug Something isn't working label Jun 19, 2020
@strijm
Copy link
Collaborator

strijm commented Jun 24, 2020

Dit issue is m.i. onterecht. Zie VNG-Realisatie/Haal-Centraal-common#44.
woonplaats wordt hier als een soort alias gebruikt, wat volgens de NL API Strategie niet mag.

@fsamwel
Copy link
Collaborator Author

fsamwel commented Jun 25, 2020

Zie mijn opmerking in VNG-Realisatie/Haal-Centraal-common#44. We kunnen dit issue in ieder geval parkeren tot we een beslissing hebben genomen over dat issue.

@strijm
Copy link
Collaborator

strijm commented Jul 1, 2020

Voor nu hebben we afgesproken, dat we alleen expliciete fieldnamen ondersteunen en geen 'aliassen'. Dus als er een property woonplaats is en een _link.woonplaats, dan kunnen deze beide worden opgegeven en dan geven we beide terug. Het opgeven van fields=woonplaats zal voorlopig niet resulteren in het leveren van zowel property woonplaats als _links.woonplaats. Na de livegan zal dit nader besproken worden zoals aangegeven in: VNG-Realisatie/Haal-Centraal-common#44.

@NicoleKortoomsBAG
Copy link
Contributor

@fsamwel @JohanBoer @MelvLee @CathyDingemanse
Naar aanleiding van de reacties op deze story bij de refinement met het team wil ik het graag nog eens hierover hebben.

@NicoleKortoomsBAG
Copy link
Contributor

@fsamwel
Dit issue kan denk ik gesloten worden. Na overleg in september is geconstateerd dat deze feature voor BAG onnodig ingewikkeld is. Afgesproken is dat de featurebeschrijvingen voor BAG, BRK en BRP niet gelijk worden gemaakt op dit punt en BAG zo simpel mogelijk wordt gehouden. Dit betekende een extra feature binnen expand en fields voor BRK en BRP omdat het daar wel een zinvolle feature is.

@fsamwel fsamwel closed this as completed Mar 4, 2021
Haal-Centraal-BAG automation moved this from To Do (kan na eerste oplevering) to Done Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

3 participants