fastapi
fastapi copied to clipboard
tiangolo
Add complete Async SQL (Relational) Databases example
This pull request adds read_one_note(), update_note(), and delete_note() endpoints to the existing Async SQL (Relational) Databases example. Which gives it a complete CRUD (Create, Read, Update, and Delete) API example.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
cf73051) 100.00% compared to head (afbe7c0) 100.00%. Report is 1036 commits behind head on master.
:exclamation: Current head afbe7c0 differs from pull request most recent head 939f4ce. Consider uploading reports for the commit 939f4ce to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## master #2730 +/- ##
===========================================
Coverage 100.00% 100.00%
===========================================
Files 540 243 -297
Lines 13969 7455 -6514
===========================================
- Hits 13969 7455 -6514
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
![codecov[bot] avatar](https://avatars.githubusercontent.com/in/254?v=4 "Published by a pub.dev verified publisher"){kind=small}
📝 Docs preview for commit 77cf6de6a1a23d3809a29e485e38348b5be102ee at: https://60161dbc4b37fc357b580511--fastapi.netlify.app
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
📝 Docs preview for commit 0213807ca0380b9b7af4856f6a609af826e8804a at: https://601798ec34cec9419de08c49--fastapi.netlify.app
📝 Docs preview for commit 717c83cdff091846335f783f2f83d96361ba842d at: https://60179a9fa0d387457b521935--fastapi.netlify.app
If we are doing delete, we should add query/if for get, update, delete checking if note exists.
@ArcLightSlavik, I'm working on this.
📝 Docs preview for commit cc671073d1e718a95ee117a353d3beeb8dbed709 at: https://601834f3f368851319e7d814--fastapi.netlify.app
📝 Docs preview for commit ba61371c27ca9c2e400b2d4c210582b4434abc0c at: https://601836bcd8f5a01812088b65--fastapi.netlify.app
📝 Docs preview for commit ea4f5e5e058cada9272fe5ffe378448123985cf5 at: https://6018d161018af6009018f6df--fastapi.netlify.app
If we are doing delete, we should add query/if for get, update, delete checking if note exists.
@ArcLightSlavik, I have implemented this.
📝 Docs preview for commit df7509abe9f6d38a83a71140edf19c12263ada0d at: https://601ed526095b3208049e3a72--fastapi.netlify.app
📝 Docs preview for commit b5bf2ba96c2e035735c5074db1077dc0d33e9031 at: https://60372739847e3d1d4366340c--fastapi.netlify.app
📝 Docs preview for commit afbe7c00b86c43b9c2d949ff6967585dfcd67eb6 at: https://60419680ec518b41a43f8290--fastapi.netlify.app
📝 Docs preview for commit 536e70df89c7153594b684e66a4288499848758b at: https://631e13a6ce733f48f266dc72--fastapi.netlify.app
📝 Docs preview for commit 8e4949f40c58bc94f0d10af98365a2db071ad386 at: https://638366e1adf1a075ff919d54--fastapi.netlify.app
📝 Docs preview for commit 4c8dd5775bf7ef82e38dc7201181452925b0b1f0 at: https://639ce97657a0d3029a1b5f82--fastapi.netlify.app
📝 Docs preview for commit 31df89e4fc4768a4708bd1e39fefd2bf8fe22e16 at: https://63ae7f12ca903426a00c6181--fastapi.netlify.app
@tiangolo I made some changes to make tests pass. Also I merged this branch with master branch.
📝 Docs preview for commit 939f4ce3c742a351c49ae54f5e438f99eb90040f at: https://63af3194f850ec21742531fe--fastapi.netlify.app