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.

Sign up for GitHub

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

Jump to bottom

📝 Add External Link: Authorization on FastAPI with Casbin #5712

Merged
merged 3 commits into from Jan 7, 2023
Merged

📝 Add External Link: Authorization on FastAPI with Casbin #5712

merged 3 commits into from Jan 7, 2023

Conversation

Xhy-5000
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

📝 Docs preview for commit c6f686d at: https://638641bc3645c4087b80fbad--fastapi.netlify.app

@Xhy-5000 Xhy-5000 mentioned this pull request Dec 1, 2022
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2022

📝 Docs preview for commit 752427e at: https://638833491d7ad53cec55563b--fastapi.netlify.app

@hsluoyz
Copy link

hsluoyz commented Dec 7, 2022

@iudeen plz review

@hsluoyz hsluoyz mentioned this pull request Dec 7, 2022
Closed
iudeen
iudeen approved these changes Dec 7, 2022
View reviewed changes
Copy link
Contributor

@iudeen iudeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

I see this is added. And the article is relevant to FastAPI.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 5634311 at: https://639ce52b18290c0968ff4878--fastapi.netlify.app

@hsluoyz
Copy link

hsluoyz commented Dec 17, 2022

@tiangolo plz review

@tiangolo tiangolo changed the title Add Casbin x FastAPI Tutorial to External Links section 📝 Add External Link: Authorization on FastAPI with Casbin Jan 7, 2023
@tiangolo
Copy link
Owner

tiangolo commented Jan 7, 2023

Great, thanks @Xhy-5000! ☕

Thanks @iudeen for the clear and explicit review! 🤓

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2023

📝 Docs preview for commit e4e3eaa at: https://63b9891809215a25781e291d--fastapi.netlify.app

@tiangolo tiangolo merged commit 27ce2e2 into tiangolo:master Jan 7, 2023
JonasKs pushed a commit to JonasKs/fastapi that referenced this pull request Jan 12, 2023
@Xhy-5000 @tiangolo @JonasKs
📝 Add External Link: Authorization on FastAPI with Casbin (tiangolo#5712
6812edf 
)

Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
axel584 pushed a commit to axel584/fastapi that referenced this pull request Mar 5, 2023
@Xhy-5000 @tiangolo @axel584
📝 Add External Link: Authorization on FastAPI with Casbin (tiangolo#5712
3b1490b 
)

Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
mahenzon pushed a commit to mahenzon/fastapi that referenced this pull request Mar 11, 2023
@Xhy-5000 @tiangolo @mahenzon
📝 Add External Link: Authorization on FastAPI with Casbin (tiangolo#5712
b5993b0 
)

Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iudeen iudeen iudeen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Xhy-5000 @hsluoyz @tiangolo @iudeen