-
Notifications
You must be signed in to change notification settings - Fork 11
Added /core/date-time
#252
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
base: develop
Are you sure you want to change the base?
Conversation
TimvdLippe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ziet er goed uit! 2 kleine wijzigingen, de rest lijkt mij voldoende om te bespreken in het TO.
|
Willen we misschien ook nog bepaalde namen van velden vastleggen? Dus als er een datum is, dan moet het veld "date", "datum", "_date" of "_datum" heetten. En als het een timestamp is, dan moet het veld "timestamp", "tijdstip", "_timestamp" of "*_tijdstip" heetten. |
Co-authored-by: Tim van der Lippe <TimvdLippe@users.noreply.github.com>
TimvdLippe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nog wat kleine fixes. De andere vragen moeten we offline even bespreken wat we daar mee willen.
|
https://swagger.io/docs/specification/v3_0/data-models/data-types/#strings geeft aan dat openapi een Tevens lijkt RFC 9557 een restrictie te zijn op ISO8601 waarbij er minder mogelijkheden te zijn. Deze RFC is relevant voor internet protocollen, dus lijkt het beter om deze RFC te gebruiken ipv de ISO standaard. Wat betreft de veldnamen laten we dat voor nu buiten de regel. We gaan dit op het TO bespreken of hier alsnog een restrictie op moet plaatsvinden. De linter check laten we nog even achterwege, omdat we niet goed weten hoe je dit geautomatiseerd kan testen als we enkel een |
Co-authored-by: Tim van der Lippe <TimvdLippe@users.noreply.github.com>
|
Hi Tim, in de concept ADR staat dat alle datetimes het ISO 8601 format YYYY-MM-DDTHH:mm:ssZ moeten hebben. Ik vraag me af of dit handig is. De Z betekent Zulu time, ofwel UTC+00:00. Met de keuze voor dit format verplicht je dus iedereen alle data-tijden om te zetten in timezone +:00:00 (wat in de wintertijd 1 uur en in de zomertijd 2 uur verschilt met onze tijd) en weer terug. Het is misschien logischer (zeker met toepassingen die vooral in de Nederlandse context gebruikt worden) om het format YYYY-MM-DDTHH:mm:ss**+/-hh:mi** ook toe te staan of zelfs voor te schrijven Verder detail: volgens mij schrijf je de uren als hh, niet als HH (zie https://www.w3.org/TR/NOTE-datetime). |
Op basis van review feedback.
|
Naar mijn weten is er geen verdere feedback van TO-leden tot nu toe, dus ik heb het label "Ter goedkeuring" toegevoegd voor bespreking op het TO van 2 december. Het voorstel vanuit TO-leden is om tijdens het TO deze wijziging vast te stellen, maar daarna nog een consultatie te doen specifiek bij het Kennisplatform API's. Op basis van de feedback kunnen we dan in het eerste TO van 2026 de definitieve vaststelling in het TO doen en deze wijziging doorvoeren. |
|
Enkele linter regels geschreven. Hierbij de resultaten: Interessante bevindingen:
Voor dat laatste punt, wat denken we van dit soort gevallen? Volgens mij hebben ze hier |
f04d381 to
bc7dd2d
Compare
bc7dd2d to
8afad81
Compare
|
Deze wijziging is goedgekeurd bij het TO 2025-12-02 voor consultatie bij Kennisplatform API's. Dat houdt in dat we het hoofdstuk "Date & Time" als geheel consulteren. Tevens was er nog een behoefte aan een aparte regel dat als er zowel een tijd als datum veld apart in een response zitten, die moeten worden samengevoegd tot 1 veld. Daarnaast nog een noot toevoegen dat er goed moet worden nagedacht als er enkel |
Voorzet voor regel voor datum en tijd
closes #195