-
Notifications
You must be signed in to change notification settings - Fork 27
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
Oplossen fout in OAS m.b.t. expand #2427
Conversation
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.
edit: zie bijgewerkte review hieronder
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.
Het volgende heb ik gecheckt:
- onterecht zichtbare '_expand'-property by POST-operaties op 'rollen' en 'zaakobjecten' is verdwenen.
- voorbeeldresponses bij GET-operaties op zaken tonen expanded resources 'rollen' en 'zaakobjecten' op de verwachte manier.
Als dit de enige twee binnen deze PR bedoelde wijzigingen zijn, zijn deze akkoord.
Gezien het feit dat deze wijziging het met zich meebrengt dat onterecht opgenomen property's zijn verwijderd, ligt een nieuwe patchrelease van de Zaken API voor de hand.
Ook is de onterecht zichtbare '_expand'-property bij de twee GET-operaties op |
Helemaal terecht, maar hun aanwezigheid was me in de bestaande spec niet opgevallen ;) |
N.a.v. issues #2412 en #2414. In de resources
rollen
enzaakobjecten
komt het attribuut_expand
(onterecht) voor. Dit is verwarrend want dit attribuut kan alleen indirect gebruikt worden via dezaken
resource. In deze PR wordt dit attribuut verwijderd daar waar het niet van toepassing is. Deze correctie heeft geen functionele gevolgen.Zie redoc voor de gewijzigde OAS.