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

Duplicate bevragingspaden #15

Open
KayodeBakker opened this issue Jul 26, 2021 · 4 comments
Open

Duplicate bevragingspaden #15

KayodeBakker opened this issue Jul 26, 2021 · 4 comments
Assignees

Comments

@KayodeBakker
Copy link

Ik lees de specificatie opnieuw en ik blijf me afvragen waarom we het volgende implementeren:

../tabellen/:tabelidentificatie/waarden
met als query params:

  • code
  • omschrijving
  • inclusiefbeeindigd
    en
    ../tabellen/:tabelidentificatie/waarden/:code

Dit lijkt me onnodig dubbelop dat beide paden de mogelijkheid biedt om code in te vullen. Vanuit een ander Haal Centraal gedachtengoed zou je code als query param weglaten uit het eerste pad zodat die puur zoek blijft waar de tweede puur raadpleeg blijft. Ook kan je kiezen om het tweede pad weg te laten zodat alles in 1 zit.

Kan iemand uitleggen waarom we deze keuze hebben gemaakt? Ik kan dit namelijk niet kan terugvinden.

@KayodeBakker
Copy link
Author

@JohanBoer @MelvLee @melsk-r @CathyDingemanse @fsamwel
Ik kan niet assignen dus dan maar zo :)

@melsk-r
Copy link
Contributor

melsk-r commented Jul 27, 2021

@KayodeBakker ik heb je rechten gegeven zodat je dit voortaan wel kan doen.

@JohanBoer
Copy link
Contributor

Volgens mij heb je gewoon gelijk Kayo en zouden we de code als query-parameter moeten verwijderen.

@JohanBoer
Copy link
Contributor

Pull request klaargezet.

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

6 participants