full-stack-fastapi-template icon indicating copy to clipboard operation
full-stack-fastapi-template copied to clipboard
verified publisher icon fastapi

Changing all status codes from raw numbers to using fastapi.status (closed #1319)

Open javadzarezadeh opened this issue 8 months ago • 1 comments

I believe that using status codes via FastAPI (Starlette, actually!) instead of using raw numbers is more readable, consistent, and makes development easier.

Re-request from PR #1319.

javadzarezadeh avatar May 12 '25 16:05 javadzarezadeh

Dear @alejsdev Please add the 'refactor' label. Thanks!

javadzarezadeh avatar May 12 '25 16:05 javadzarezadeh

Thank you, @YuriiMotov. I've made the changes you requested. I even changed the numbers in ranges to starlette HTTP codes. If you still think that numbers are better for ranges. Let me know. I'll revert it quickly.

javadzarezadeh avatar Sep 04 '25 18:09 javadzarezadeh

This pull request has a merge conflict that needs to be resolved.

github-actions[bot] avatar Sep 20 '25 16:09 github-actions[bot]

Thanks! Actually, recently we had to move some of them that were using the names from status to use the numbers, because the name was changed in a recent version of Starlette. So, for now, I'll pass on this one, but thanks! :coffee:

tiangolo avatar Sep 20 '25 17:09 tiangolo