-
Notifications
You must be signed in to change notification settings - Fork 2
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
parameter inclusiefbeeindigd moet zijn inclusiefBeeindigd #25
Comments
Beide specificeren is denk ik niet netjes. Ik zou |
Dit ligt toch genuanceerder ? Volgens mij beschrijven we path-parameters in lowerCase. Zo is dat in ieder geval bij de BAG. Niet meer relevant voor Persoon en Historie bevragen. |
ooit zijn we begonnen met alles in de url, inclusief queryparameters, in lower case te zetten. Dat was toen de stand van de NL API design rules. Later is dat aangepast in de design rules, zie https://docs.geostandaarden.nl/api/API-Strategie-ext/#query-parameters Dat hebben we gedeeltelijk opgelost, bijv. in v1.3 van de BRP Personen API (niet meer zichtbaar in v2.0). |
Toch doen. Er zijn nog geen actieve gebruikers op v2.0, release is nog niet definitief. |
@JohanBoer voer jij dit t.z.t. door? |
Ik neem haast aan dat deze issue bij BRP-Bevragen thuishoort en niet bij BRP-tabellen-bevragen... Gaarne even bevestigen |
Inmiddels toch gevonden. Sorry voor de verwariing |
we schrijven parameters in camelCase.
@CathyDingemanse dit is een breaking change. Toch doen?
@MelvLee of kunnen/moeten we dit zo specificeren dat zowel inclusiefbeeindigd als inclusiefBeeindigd toegestaan zijn?
The text was updated successfully, but these errors were encountered: