-
Notifications
You must be signed in to change notification settings - Fork 2
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider using Django 4.1+ Async ORM #1
Comments
This will be great improvement. Thanks for this idea. |
You'll want to update the minimum supported Django version, but I suspect most users using this project will want to use as much |
Since this project uses Django directly, I would recommend adding Django to the
You probably want to add By explicitly listing the dependencies you interact directly, this allows you to control minimum versions you support so you can rely on features like this. |
Note that I have run into a couple issues related to Channels and the Async ORM: So this may be something that is best deferred until the Async ORM matures and is more widely used. |
I believe that the
@database_sync_to_async
used within this project could be removed by using the Async ORM Interface introduced in Django 4.1.The text was updated successfully, but these errors were encountered: