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

4.2.5 expand=true #285

Closed
mevdschee opened this issue Mar 25, 2019 · 4 comments
Closed

4.2.5 expand=true #285

mevdschee opened this issue Mar 25, 2019 · 4 comments

Comments

@mevdschee
Copy link

Ik heb een suggestie voor een verbetering/verduidelijking. In 4.2.5 staat:

Als expand=true wordt meegegeven, dan worden alle geneste resources geladen

Dit snap ik niet, het datamodel is namelijk geen boom (kan cycles bevatten). Daarna staat:

verplicht om te specificeren welke resources en zelfs welke velden van een resource teruggeven moeten worden.

Dit lijkt me in strijd met de eerdere zin (die over expand=true). Wat is de bedoeling hiervan? Kan dit misschien iets duidelijker?

@HenriKorver
Copy link
Contributor

In issue https://github.com/Geonovum/KP-APIs/issues/12#issuecomment-467879801 wordt voorgesteld om het expand-mechanisme te verwijderen uit de API Strategie. Ik ben het eens met dat voorstel:

  • Expand is een ingewikkeld mechanisme dat je niet zomaar even goed beschreven hebt
  • Het is de vraag of je consumer-developers wilt opzadelen met deze problematiek.

@mevdschee
Copy link
Author

Expand is een ingewikkeld mechanisme dat je niet zomaar even goed beschreven hebt

Dat blijkt, want ik begrijp het niet zoals het nu opgeschreven is.

@dvh
Copy link
Collaborator

dvh commented Mar 25, 2019

+1

jasperroes referenced this issue Apr 4, 2019
Fixes issues #12 en #71
jasperroes referenced this issue Apr 4, 2019
Fixes issue #12 en #71 door de API principes 07 en 08 te verwijderen.
@jasperroes
Copy link
Collaborator

Verwerkt.

@fterpstra fterpstra transferred this issue from Geonovum/KP-APIs Sep 4, 2020
@jasperroes jasperroes transferred this issue from Logius-standaarden/API-Design-Rules Nov 25, 2020
@jasperroes jasperroes transferred this issue from another repository Nov 25, 2020
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

No branches or pull requests

4 participants