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

Snake case name convertor missmatch with openapi #652

Closed
wants to merge 1 commit into from

Conversation

Anry7794
Copy link

Using nameConvertor to snake case is leading to missmatch with open api.

If you have good annotations f.e. property $fullName in response DTO openapi will generate { fullName: "string" } but reality with nameConvertor is { full_name: "John Doe" }. So there should be fix on side of openapi or removing nameConvertor

Using nameConvertor to snake case is leading to missmatch with open api.

If you have good annotations f.e. property $fullName in response DTO openapi will generate { fullName: "string" } but reality with nameConvertor is { full_name: "John Doe" }. So there should be fix on side of openapi or removing nameConvertor
@f3l1x
Copy link
Member

f3l1x commented Jan 10, 2025

I think it's OK by me. For example streetNumber should be street_number from my point of view. I use fullname not fullName. Thus, in your case, remove CamelCaseToSnakeCaseNameConverter in your app.

@f3l1x f3l1x closed this Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants