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

Allow content-type application/merge-patch+json #1348

Open
uga-rosa opened this issue Nov 23, 2024 · 0 comments
Open

Allow content-type application/merge-patch+json #1348

uga-rosa opened this issue Nov 23, 2024 · 0 comments
Labels
enhancement New feature or request openapi-features OpenAPI features support issues

Comments

@uga-rosa
Copy link

Description

First, thank you for a great generation tool!

In a patch request, there is a meaningful difference between setting a field to null and a field being missing, but handling this in Go can be challenging. I was looking for projects that are seriously addressing this issue.

https://ogen.dev/docs/types/optional/

However, despite such excellent efforts, I realized that it is not possible to use the content-type application/merge-patch+json.

The internal processing should be almost the same as JSON, so the necessary implementation can be kept to a minimum. I would appreciate your support.

References

@uga-rosa uga-rosa added enhancement New feature or request openapi-features OpenAPI features support issues labels Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request openapi-features OpenAPI features support issues
Projects
None yet
Development

No branches or pull requests

1 participant