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.

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

Readme example of bindings within celery tasks example, Support for channels v1.x #43

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mattyg
Copy link

@mattyg mattyg commented Feb 19, 2018

Thanks so much for this project! It was extremely helpful and the missing piece I was searching for!

I updated the Readme to include how to get model bindings to work with model instances within celery tasks.

I also updated the Reame and requirements.txt to indicate support for only channels 1.x, until #42 is resolved.

@mattyg mattyg changed the title Update readme to include bindings within celery tasks example, Update support for channels v1.x Readme example of bindings within celery tasks example, Support for channels v1.x Feb 19, 2018
@Herst
Copy link
Contributor

Herst commented Feb 19, 2018

Maybe this can then be the basis for the last proper release for channels 1.x.

@linuxlewis
Copy link
Owner

I would move the celery example out of the getting started. It's technically not required. I've fixed the package requirements on master

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.

3 participants