Skip to content

📝 Update usage of Token model in security docs#9313

Merged
alejsdev merged 5 commits intofastapi:masterfrom
piotrszacilowski:fix-token-model-usage-in-docs
Jan 11, 2024
Merged

📝 Update usage of Token model in security docs#9313
alejsdev merged 5 commits intofastapi:masterfrom
piotrszacilowski:fix-token-model-usage-in-docs

Conversation

@piotrszacilowski
Copy link
Contributor

In the documentation regarding security in the code were created Token model, while it was not used. I have fixed it.

@github-actions
Copy link
Contributor

@piotrszacilowski
Copy link
Contributor Author

anyone willing to review?

@tiangolo tiangolo added the docs Documentation about how to use FastAPI label Oct 2, 2023
@alejsdev alejsdev changed the title Fix usage of Token model in security docs 📝 Update usage of Token model in security docs Jan 11, 2024
@tiangolo
Copy link
Member

@alejsdev
Copy link
Member

Thanks for your help! @piotrszacilowski 🙂

The Token was actually used in the response_model but indeed it was better to return an instance of it.

I also updated the response model to use the Token as a return type and updated the highlighted lines in the docs.

Thanks 😸

@alejsdev alejsdev merged commit fd97e8e into fastapi:master Jan 11, 2024
nilslindemann pushed a commit to nilslindemann/fastapi that referenced this pull request Jan 13, 2024
nilslindemann pushed a commit to nilslindemann/fastapi that referenced this pull request Jan 13, 2024
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.

3 participants