#31949: Allow builtin view decorators to be applied directly to async views.
--------------------------------+-------------------------------------
     Reporter:  Michael Galler  |                    Owner:  Ben Lomax
         Type:  New feature     |                   Status:  assigned
    Component:  Core (Other)    |                  Version:  3.1
     Severity:  Normal          |               Resolution:
     Keywords:  async           |             Triage Stage:  Accepted
    Has patch:  1               |      Needs documentation:  0
  Needs tests:  0               |  Patch needs improvement:  1
Easy pickings:  0               |                    UI/UX:  0
--------------------------------+-------------------------------------

Comment (by Ben Lomax):

 Hi Carlton,

 Thanks for that clarification, I've now updated the PR to:

 1. Remove the `sync_async_wrapper` function completely; all the decorators
 now handle their logic "inline". I've still pulled out some of the shared
 code between the sync and async versions of the decorators, so you'll
 still see a few `_process_request` and `_process_response` about the
 place, but it's now much clearer what they actually do.

 2. Make all of the `contrib.auth` decorators be able to handle sync and
 asycn views.

 The documentation update still points to version 4.2, let me know if
 that's no longer correct.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/31949#comment:25>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070184d8cc1e55-92b4a2e2-d9df-41bc-aee2-6dbbbb4512ca-000000%40eu-central-1.amazonses.com.

Reply via email to