-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
✨ Add support for Pydantic models to validate form fields #2431
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
✨ Add support for Pydantic models to validate form fields #2431
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2431 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 242 242
Lines 7396 7404 +8
=========================================
+ Hits 7396 7404 +8
Continue to review full report at Codecov.
|
|
📝 Docs preview for commit 3cfa1961afe4e3e4f46df940fae4d599f150b419 at: https://5fc17464d55272ec05defe54--fastapi.netlify.app |
|
📝 Docs preview for commit 6b0cbef9e7008758c694f44b0855a9faaf529d18 at: https://5fc1773dc2e60836fe4d341e--fastapi.netlify.app |
|
📝 Docs preview for commit d37c9e70b7ed3c8c0c81fc2578cf05e90ee15708 at: https://5fc52a0042cac00fa20ee566--fastapi.netlify.app |
|
There's something wrong with the number of changes here 🤔 |
d37c9e7 to
208a60e
Compare
@Kludex Don't know why it happened, I just merged latest master in. Now I rebased it, looks fine now. |
|
📝 Docs preview for commit 208a60e at: https://5fc6d52ff74c008bcc0d44eb--fastapi.netlify.app |
|
@Kludex could you do the CR for me please? |
|
Thanks for the patience with my reply! 😅 This was covered in #12129, it is available in FastAPI 0.113.0 🎉 The new docs: https://fastapi.tiangolo.com/tutorial/request-form-models/ Given that, I'll close this one, but thanks for the effort! 🍰 |
This should fix #2387 and #2365