Skip to content

Drop channels support #336

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

Merged
merged 1 commit into from
Jun 16, 2023
Merged

Drop channels support #336

merged 1 commit into from
Jun 16, 2023

Conversation

zerolab
Copy link
Member

@zerolab zerolab commented Jun 16, 2023

Builds on #334
Fixes #317

@zerolab zerolab changed the base branch from main to feat/flit June 16, 2023 15:22
@zerolab zerolab force-pushed the chore/drop-channels branch from d832f87 to 3847ba8 Compare June 16, 2023 15:30
@zerolab zerolab requested a review from dopry June 16, 2023 15:50
@dopry
Copy link
Collaborator

dopry commented Jun 16, 2023

LGTM!

Copy link
Collaborator

@dopry dopry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Neither you nor I are testing this. There is enough volatility around asgi and channels at the moment to make maintaining it beyond our current capabilities in terms of our availability. The vast majority of cms driven sites really just need an API for headless.

I'm good with dropping this.

Base automatically changed from feat/flit to main June 16, 2023 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop Subscriptions (i.e. official channels/websocket support)
2 participants