Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

started on channels 2 updates (WIP) #9

Closed
wants to merge 5 commits into from
Closed

started on channels 2 updates (WIP) #9

wants to merge 5 commits into from

Conversation

colanconnon
Copy link
Contributor

Starting to update the django channels to use channels 2

@colanconnon
Copy link
Contributor Author

colanconnon commented Feb 19, 2018 •
edited
Loading

TODO:

  • Update current code to work with new sync consumers

  • Not working in chrome but works in Safari and FF

  • Allow for async consumers alongside sync

  • Add tests

@SmileyChris
Copy link
Contributor

Hi @colanconnon -- check out my take on this in PR #18

@xeor xeor mentioned this pull request Dec 3, 2018
Closed
@colanconnon
Copy link
Contributor Author

Closing in favor of other pr

@colanconnon colanconnon closed this Dec 4, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@colanconnon @SmileyChris