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

📝 Replace HTTP code returned in case of existing user error in docs for testing #4482

Merged
merged 4 commits into from
Jan 9, 2024

Conversation

TristanMarion
Copy link
Contributor

@TristanMarion TristanMarion commented Jan 26, 2022

Change the HTTP code from 400 to 409 to indicate a Conflict

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Change the HTTP code from 400 to 409 ton indicate a Conflict
@TristanMarion TristanMarion force-pushed the fix/use-409-testing-doc branch from 6e8dd24 to b48b47d Compare January 26, 2022 14:54

Verified

This commit was signed with the committer’s verified signature.
crazy-max CrazyMax
@codecov
Copy link

codecov bot commented Sep 11, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cf73051) 100.00% compared to head (717d7a5) 100.00%.
Report is 874 commits behind head on master.

❗ Current head 717d7a5 differs from pull request most recent head 8ba9a0e. Consider uploading reports for the commit 8ba9a0e to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4482   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          540       540           
  Lines        13969     13951   -18     
=========================================
- Hits         13969     13951   -18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 717d7a5 at: https://631e1329ce733f48c566dcfd--fastapi.netlify.app

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@github-actions
Copy link
Contributor

📝 Docs preview for commit 5c93713 at: https://639cd46dc3d3b819c31d1cda--fastapi.netlify.app

@tiangolo tiangolo added the docs Documentation about how to use FastAPI label Oct 2, 2023
@tiangolo tiangolo changed the title docs: replace HTTP code returned in case of existing user 📝 Replace HTTP code returned in case of existing user error in docs for testing Jan 9, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@tiangolo
Copy link
Member

tiangolo commented Jan 9, 2024

Nice, thank you @TristanMarion! 🍰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation about how to use FastAPI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants