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

Oplossen fout in OAS m.b.t. expand #2427

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

HenriKorver
Copy link
Collaborator

@HenriKorver HenriKorver commented Mar 12, 2024

N.a.v. issues #2412 en #2414. In de resources rollen en zaakobjecten komt het attribuut _expand (onterecht) voor. Dit is verwarrend want dit attribuut kan alleen indirect gebruikt worden via de zaken 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.

Copy link
Collaborator

@hdksi hdksi left a 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

Copy link
Collaborator

@hdksi hdksi left a 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.

@HenriKorver
Copy link
Collaborator Author

HenriKorver commented Mar 12, 2024

Ook is de onterecht zichtbare '_expand'-property bij de twee GET-operaties op rollen en zaakobjecten verwijderd. Immers expand op deze twee resources mag alleen via de zaken resource plaatsvinden.

@hdksi
Copy link
Collaborator

hdksi commented Mar 12, 2024

Ook is de onterecht zichtbare '_expand'-property bij de twee GET-operaties op rollen en zaakobjecten verwijderd. Immers expand op deze twee resources mag alleen via de zaken resource plaatsvinden.

Helemaal terecht, maar hun aanwezigheid was me in de bestaande spec niet opgevallen ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants