Skip to content

ASGI message endpoint#121

Draft
adamghill wants to merge 1 commit intomainfrom
asgi
Draft

ASGI message endpoint#121
adamghill wants to merge 1 commit intomainfrom
asgi

Conversation

@adamghill
Copy link
Member

No description provided.

@adamghill adamghill self-assigned this Jan 23, 2021
@adamghill adamghill marked this pull request as draft January 28, 2021 02:19
Thembahank added a commit to Erteno/django-unicorn that referenced this pull request Apr 8, 2021
Initial work for supporting ASGI. https://github.com/adamghill/django:
Merged ASGI Branch until PR django-commons#121 is resolved
@JohananOppongAmoateng
Copy link
Contributor

I am curious what under changes we need to make to fully support async

@adamghill
Copy link
Member Author

I am curious what under changes we need to make to fully support async

Off the top of my head:

Although, would need to dig into this more. Personally, I'd like to understand if the code complexity is worth the effort.

@JohananOppongAmoateng
Copy link
Contributor

Well i think the code will be a bit complex

@JohananOppongAmoateng
Copy link
Contributor

I am curious what under changes we need to make to fully support async

Off the top of my head:

Although, would need to dig into this more. Personally, I'd like to understand if the code complexity is worth the effort.

well cant we use sync_to_async for all the currently sync sections of django? well i think some models methods in the orm have async support? I mean just thinking aloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants