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

OAS specificatie van BAG Bevragen gebruikt definities van Common 1.1.0 #77

Open
MelvLee opened this issue Apr 6, 2021 · 5 comments
Open
Assignees

Comments

@MelvLee
Copy link
Collaborator

MelvLee commented Apr 6, 2021

Dit zorgt ervoor dat de examples van de 4xx en 5xx responses een foute url bevatten

@JohanBoer JohanBoer self-assigned this Apr 6, 2021
@JohanBoer
Copy link
Collaborator

Ga ik aanpassen.

@fsamwel
Copy link
Collaborator

fsamwel commented Apr 6, 2021

@JohanBoer denk je eraan dat we hebben afgesproken dat de BAG beheerders de genereerversie beheren vanuit hun implementatie? Die moet je dus niet zelf wijzigen, maar door BAG worden gewijzigd (als het probleem daarin ook speelt).

@JohanBoer
Copy link
Collaborator

@JohanBoer denk je eraan dat we hebben afgesproken dat de BAG beheerders de genereerversie beheren vanuit hun implementatie? Die moet je dus niet zelf wijzigen, maar door BAG worden gewijzigd (als het probleem daarin ook speelt).

Dat had ik niet meer paraat.
Deze examples komen ook in de genereer-variant terecht. Dus er is wel impact op de genereer-variant.
Dan moeten we ook zorgen dat de lint-oas op de BAG-bevragen repository wordt aangepast. Daarin wordt nu automatisch de genereer-versie geresolved zoals dat ook in andere repositories geregeld is. @MelvLee zou jij dat willen aanpassen op de BAG-bevragen Github-repo ?

@fsamwel
Copy link
Collaborator

fsamwel commented Apr 6, 2021

dat hoeft nu niet. in develop is dit geen probleem en juist handig, want zo kunnen we die gegenereerde specs gebruiken in Swagger.

Alleen wanneer je naar master gaat moet ook bij BAG iets worden gedaan. De afspraak is dacht ik dat aan het einde van een release het BAG beheer team deze aanpassing doet.

@JohanBoer
Copy link
Collaborator

Inmiddels alle verwijzingen naar versie 1.2.0 van de common in develop aangebracht. Ook in de verwijzingen naar de common features.

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

No branches or pull requests

3 participants