#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 Carlton Gibson):

 Hey Ben.

 I think the shorter version would be that the `sync_async_wrapper` strikes
 me as overly complex. I think it would be much easier on the eye inlining
 the requisite code each time. I suspect for each decorator the needed code
 is minimal. That wouldn't be as DRY™, but it would exhibit better locality
 of behaviour, and be easier to maintain.

 An incremental way forward would be looking at a subset of the decorators.
 I suspect, the `contrib.auth` decorators are the ones I'm most likely to
 want to use first... 🤔

-- 
Ticket URL: <https://code.djangoproject.com/ticket/31949#comment:23>
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/01070184a9dbdb1c-1026cf24-3e04-484f-a85b-c6a9f9e63b79-000000%40eu-central-1.amazonses.com.

Reply via email to