#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:  0               |      Needs documentation:  0
  Needs tests:  0               |  Patch needs improvement:  0
Easy pickings:  0               |                    UI/UX:  0
--------------------------------+-------------------------------------

Comment (by Ben Lomax):

 Hi Carlton, and a Happy New Year to you too 😁

 Splitting up the work and moving the tests to separate files seems like a
 good idea to me, I'll try to get started on those in the next few weeks at
 the latest. The one caveat I would put forward is to keep the
 `test_decorators_sync_and_async_capable` test
 
([https://github.com/LomaxOnTheRun/django/blob/65bf55b643300f5f314be691f3cbd96134c7f1d7/tests/decorators/tests.py#L496
 link to code]) in the shared test file, as it's less of a test of the
 behaviour of each decorator, and in theory shouldn't change for a while
 anyway (i.e. none of them should go back to just being only sync or async
 capable). How does that sound?

-- 
Ticket URL: <https://code.djangoproject.com/ticket/31949#comment:28>
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018592544eca-fba4a1c6-98f9-4c41-ad52-7bdb60e61817-000000%40eu-central-1.amazonses.com.

Reply via email to