Als developer wil ik het agenda component common ground proof maken - Githubissues

VNG-Realisatie / huwelijksplanner

Other

4 stars 4 forks source link

Als developer wil ik het agenda component common ground proof maken #218

Open rubenvdlinde opened 4 years ago

rubenvdlinde commented 4 years ago

Korte samenvatting -> component door de wasstraat halen Taken lijst todo:

[ ] Architectuur afstemming
[x] Ontwerp gegevens model
[x] Realisatie
[ ] Postman testscripts
[x] Documentatie Redoc
[ ] Testscripts aan leveren aan testvoorziening
[x] Voorbeeld implementatie
[ ] Code opleveren in Utrecht repo

Checklist Definition of done

[x] Oas documentatie
[ ] Postman scripts (test scripts)
[ ] Aangeleveren testscripts aan de api test voorziening
[x] Containers moeten gepubliceert zijn
[ ] Gereviewd op de design decisions
[x] Voorbeeld implementatie beschikbaar
[x] Redoc moet beschikbaar zijn
[ ] Documentatie moet beschikbaar zijn op github-pages
[x] NLX integratie moet werken

matthiasoliveiro commented 4 years ago

Checklist DoD toegevoegd

matthiasoliveiro commented 4 years ago

Subtasks toegevoegd

matthiasoliveiro commented 4 years ago

@CMasselink Zou je deze kunnen reviewen: https://arc.dev.huwelijksplanner.online/

CMasselink commented 4 years ago

@matthiasoliveiro @rubenvdlinde Zou er nog gekeken kunnen worden naar de documentatie / OAS. De toelichting bij attributen klopt niet: afbeelding

matthiasoliveiro commented 4 years ago

Dit is een bug en zouden we zo moeten behandelen.

matthiasoliveiro commented 4 years ago

Zou nu opgelost moeten zijn: https://ac.huwelijksplanner.online/

CMasselink commented 4 years ago

Nog niet helemaal goed:

Coused moet sowieso caused zijn, maar is triggered niet een betere vertaling?

rubenvdlinde commented 4 years ago

Dit heeft te maken met interne verwijzingen van de swagger documentatie die volgens de specs lijken te klopen maar toch fout gaan in redoc.

Een vergelijking met (pet store)[https://redocly.github.io/redoc/#operation/addPet] leverd ook op dat de code overeen komt en daar doet de redoc het wel dus deze is best wierd ik duik er even verder in.