Opened 2 years ago

Closed 2 years ago

#34147 closed New feature (invalid)

Add aall() for related managers.

Reported by: Mariusz Felisiak Owned by:
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords:
Cc: Jon Janzen, Carlton Gibson Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

aall() method (asynchronous versions of all()) should be available for related managers:

AttributeError: 'RelatedManager' object has no attribute 'aall'

Change History (6)

comment:1 by Bhuvnesh, 2 years ago

Owner: changed from nobody to Bhuvnesh
Status: new → assigned

comment:2 by Bhuvnesh, 2 years ago

May i try to prepare a patch for this one?

Last edited 2 years ago by Bhuvnesh (previous) (diff)

comment:3 by Carlton Gibson, 2 years ago

Triage Stage: Unreviewed → Accepted

comment:4 by Bhuvnesh, 2 years ago

Owner: Bhuvnesh removed
Status: assigned → new

comment:5 by Simon Charette, 2 years ago

Is there really a need for aall() given it doesn't perform any I/O but solely copy objects over?

e.g. There is no QuerySet.aall() like there is no .afilter and no BaseManager.aall?

in reply to:  5 comment:6 by Mariusz Felisiak, 2 years ago

Resolution: → invalid
Status: new → closed
Triage Stage: Accepted → Unreviewed

Replying to Simon Charette:

Is there really a need for aall() given it doesn't perform any I/O but solely copy objects over?

e.g. There is no QuerySet.aall() like there is no .afilter and no BaseManager.aall?

Right 🤦 I withdraw my request.

Note: See TracTickets for help on using tickets.
Back to Top