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

fix WOOVerzoeken typo #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ken-ll
Copy link

@ken-ll ken-ll commented Aug 24, 2023

Deze PR fixt een typo in de API specs en ItemController. Op dit moment zijn we geforceerd om WooVerzoeken verkeerd te spellen in onze API specs, omdat de ItemController specifiek naar "Wooverzoeken" op zoek is.

Wanneer we "Wooverzoeken" gebruiken in onze API specs in plaats van camelcase "WooVerzoeken" dan krijg je een WOOItemList model met verkeerde namen:

Screenshot 2023-08-24 at 17 07 11

Op dit moment lossen we dit zelf op met een patch, maar het is netter om dit in de plugin al te fixen voor toekomstige gemeentes.

Let op, als je dit aanpast is het wel een breaking change in de plugin.

Groetjes,
Ken
Level Level

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

Successfully merging this pull request may close these issues.

1 participant