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

Include ajvFilePlugin into source and follow OpenAPI convention. #443

Merged
merged 6 commits into from
Jun 18, 2023

Conversation

arthurfiorette
Copy link
Contributor

Previously, there was only one example on how to use @fastify/multipart together with @fastify/swagger. This PR includes the function in the source code to avoid forcing the programmer to always have to open this documentation to see if anything has been changed. And we can be sure that no developer will ever check this again, where future bugfixes will not be fixed for them.
Also, the previous type: file has been changed to an OpenAPI compatible type: string, format: binary.

Checklist

@arthurfiorette arthurfiorette changed the title Include ajvFilePlugin into source to follow OpenAPI convention. Include ajvFilePlugin into source and follow OpenAPI convention. May 26, 2023
README.md Outdated Show resolved Hide resolved
@arthurfiorette
Copy link
Contributor Author

@climba03003, sorry for pinging, but any updates on this pr?

@felipe-bergamaschi
Copy link

@mcollina We need this merge

@arthurfiorette
Copy link
Contributor Author

Hey @climba03003, sorry to ping again, can you give a follow up?

index.d.ts Show resolved Hide resolved
index.js Show resolved Hide resolved
@mcollina mcollina merged commit 180c291 into fastify:master Jun 18, 2023
15 checks passed
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.

None yet

4 participants