Opened 19 months ago

Closed 4 weeks ago

#21803 closed New feature (fixed)

Support post-commit hooks

Reported by: aaugustin Owned by: nobody
Component: Database layer (models, ORM) Version: master
Severity: Normal Keywords:
Cc: charettes, niels.busch@… 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

We should merge https://github.com/carljm/django-transaction-hooks once it has stabilized.

It looks robust and correctly builds on top of the transaction management code.

See also #14051.

Change History (10)

comment:1 Changed 19 months ago by charettes

  • Cc charettes added

comment:2 Changed 19 months ago by niels

  • Cc niels.busch@… added

comment:3 Changed 18 months ago by timo

  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 3 months ago by pelme

I've made an initial PR (not yet complete), with a port of django-transaction-hooks:

https://github.com/django/django/pull/4593

comment:5 Changed 3 months ago by carljm

  • Has patch set
  • Patch needs improvement set

comment:6 Changed 3 months ago by timgraham

  • Patch needs improvement unset

comment:7 Changed 3 months ago by timgraham

  • Patch needs improvement set

comment:8 Changed 4 weeks ago by timgraham

  • Patch needs improvement unset

comment:9 Changed 4 weeks ago by timgraham

  • Triage Stage changed from Accepted to Ready for checkin

comment:10 Changed 4 weeks ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 00a1d4d0:

Fixed #21803 -- Added support for post-commit callbacks

Made it possible to register and run callbacks after a database
transaction is committed with the transaction.on_commit() function.

This patch is heavily based on Carl Meyers django-transaction-hooks
<https://django-transaction-hooks.readthedocs.org/>. Thanks to
Aymeric Augustin, Carl Meyer, and Tim Graham for review and feedback.

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