Skip to content

📝 Add missing highlights in docs/en/docs/tutorial/security/oauth2-jwt.md#10995

Closed
Skyvastern wants to merge 2 commits intofastapi:masterfrom
Skyvastern:patch-1
Closed

📝 Add missing highlights in docs/en/docs/tutorial/security/oauth2-jwt.md#10995
Skyvastern wants to merge 2 commits intofastapi:masterfrom
Skyvastern:patch-1

Conversation

@Skyvastern
Copy link

In oauth2-jwt.md or the following page - https://fastapi.tiangolo.com/tutorial/security/oauth2-jwt/#update-the-dependencies the addition of TokenData class is not highlighted. So I added the highlights (in Update the dependencies section) as this was the place where TokenData class was first used. It's used in the function get_current_user.

@tiangolo
Copy link
Member

📝 Docs preview for commit 777eac4 at: https://27e1a622.fastapitiangolo.pages.dev

@alejsdev alejsdev changed the title Add missing highlights in oauth2-jwt.md 📝 Add missing highlights in docs/en/docs/tutorial/security/oauth2-jwt.md Jan 19, 2024
@alejsdev alejsdev added the docs Documentation about how to use FastAPI label Jan 19, 2024
@tiangolo
Copy link
Member

tiangolo commented Feb 5, 2024

📝 Docs preview for commit d2e1bc1 at: https://022e4cfc.fastapitiangolo.pages.dev

@tiangolo
Copy link
Member

tiangolo commented Feb 8, 2024

Thanks @Skyvastern! But the highlights are there as I intended, to focus on the specific function that the text is referring to. So I'll pass on this one, but thanks for the effort! ☕

@tiangolo tiangolo closed this Feb 8, 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