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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摑 Add notification message warning about old versions of FastAPI not supporting Annotated #9298

Merged
merged 3 commits into from
Apr 13, 2023

Conversation

grdworkin
Copy link
Contributor

The new annotated dependency feature creates some to unexpected errors on old versions:

#9277
#9297

This adds in a quick warning to the dependencies tutorial for anyone about to integrate the new functionality.

Feel free to close this request without discussion if you don't think it will be beneficial. It won't bother me

@tiangolo tiangolo changed the title Update dependency docs with warning for old versions of FastAPI 馃摑 Add notification message warning about old versions of FastAPI not supporting Annotated Apr 13, 2023
@tiangolo tiangolo enabled auto-merge (squash) April 13, 2023 18:54
@tiangolo
Copy link
Owner

Great, thanks! I updated the message a bit and added it in the first two places in the docs where it is used. 馃 馃嵃

@github-actions
Copy link
Contributor

馃摑 Docs preview for commit c3e7649 at: https://64385078a4c56107710d5d02--fastapi.netlify.app

@tiangolo tiangolo merged commit 8df8630 into tiangolo:master Apr 13, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants