-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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 documentation on how to use the Django ORM inside FastAPI #11782
base: master
Are you sure you want to change the base?
Conversation
patrick91
commented
Jul 1, 2024
- Add initial documentation for using the Django ORM
- Add async example
📝 Docs preview for commit 1b83a5d at: https://89c6eacf.fastapitiangolo.pages.dev |
📝 Docs preview for commit 85ea9ae at: https://70478e6c.fastapitiangolo.pages.dev |
📝 Docs preview for commit 5909e93 at: https://1421e46e.fastapitiangolo.pages.dev |
📝 Docs preview for commit e639870 at: https://cbdcbcd0.fastapitiangolo.pages.dev |
📝 Docs preview for commit 048a762 at: https://5ee78811.fastapitiangolo.pages.dev |
048a762
to
63212e6
Compare
📝 Docs preview for commit 3768b3c at: https://3f3e5c23.fastapitiangolo.pages.dev |
📝 Docs preview for commit d259c13 at: https://ee45910f.fastapitiangolo.pages.dev |
📝 Docs preview for commit 5e49b3f at: https://1fc2129e.fastapitiangolo.pages.dev |
I think it would be better to move this guide to "How To - Recipes" section. Also, using global object to access DB session is not a good practice. It should be at least moved to dependency to allow it to be overridden in tests. Overall, I like the manner of presentation, and I find this guide useful. |