Opened 2 years ago

Last modified 2 years ago

#34135 closed New feature

Add async interface to ManyRelatedManager & RelatedManager — at Version 5

Reported by: Jon Janzen Owned by: Jon Janzen
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords: async
Cc: Carlton Gibson Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Jon Janzen)

With #34112 and #33646 combined, there is now an async interface for reading/writing Models in a lot of cases. One of the obvious places where that isn't true is in RelatedManager instances. RelatedManager instances are missing async versions of:

  • add
  • remove
  • clear
  • set

And this might be a separate ticket but the docs (1) are missing a reference to acreate which according to dir() exists on ManyRelatedManager

(1) ​https://docs.djangoproject.com/en/dev/ref/models/relations/#django.db.models.fields.related.RelatedManager

Change History (5)

comment:1 by Jon Janzen, 2 years ago

FWIW I'm also willing to work on a PR for this ticket, assuming a similar solution to #34112.

(apologies if this ticket is not well-formed or I missed another ticket that covers the same topics)

comment:2 by Jon Janzen, 2 years ago

Description: modified (diff)

comment:3 by Mariusz Felisiak, 2 years ago

Cc: Carlton Gibson added
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → New feature

Sounds reasonable.

comment:4 by Jon Janzen, 2 years ago

Owner: changed from nobody to Jon Janzen
Status: new → assigned

comment:5 by Jon Janzen, 2 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top