-
Notifications
You must be signed in to change notification settings - Fork 1.1k
make doc corrections for django channels #2232
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
make doc corrections for django channels #2232
Conversation
docs/installation.rst
Outdated
| The Debug Toolbar currently has experimental support for async projects, | ||
| but not for Django Channels. The Debug Toolbar is compatible with the | ||
| following exceptions: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reads a bit weird with two blocks of exceptions. Maybe we just move the channels incompatibility down into the list of exceptions so it's just one list of ways the toolbar doesn't work with async projects?
docs/architecture.rst
Outdated
| are disabled by default when running in async environment. | ||
| follow the progress of this issue in `Async compatible toolbar project <https://github.com/orgs/jazzband/projects/9>`_. | ||
| - Django Channels incompatibility: The toolbar does not currently support | ||
| Django Channels' consumers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Django Channels' consumers. | |
| Django Channels' consumers due to the difference in middleware design. |
The little contextual note may be helpful for a future reader if they don't realize to check the issues first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for moving this along! My comments are nitpicky. Please also add a mention of this to the changelog (I think it's changes.rst).
|
How's it looking now? @tim-schilling |
|
Thank you @Chiemezuo! |
|
Thanks for the merge! @tim-schilling |
Description
Following up on PR #2228 , the documentation needed some correction to state that the Toolbar currently doesn't support Django Channels.
Fixes #2231
Checklist:
docs/changes.rst.