We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hi, just playing and reviewing the code and overall idea. Looks really promissing!
Please find a few findings noted when I went through the code.
fadin/fadin/app/asgi.py
Line 17 in ae8105e
Shouldn't it use django.setup() method?
django.setup()
Lines 52 to 53 in ae8105e
Why not just passing the actual session object and AnonymousUser instance to fastapi? E.g. like
AnonymousUser
request.state.session = session request.state.user = user
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Hi, just playing and reviewing the code and overall idea. Looks really promissing!
Please find a few findings noted when I went through the code.
fadin/fadin/app/asgi.py
Line 17 in ae8105e
Shouldn't it use
django.setup()
method?fadin/fadin/app/asgi.py
Lines 52 to 53 in ae8105e
Why not just passing the actual session object and
AnonymousUser
instance to fastapi? E.g. likeThe text was updated successfully, but these errors were encountered: