Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Django 1.6 #11

Open
grillermo opened this issue Nov 9, 2013 · 2 comments
Open

Django 1.6 #11

grillermo opened this issue Nov 9, 2013 · 2 comments

Comments

@grillermo
Copy link

I have not noticed anything breaking, but i'm not sure if with the new transaction management in django 1.6 do i have to make changes to this library.

@alurin
Copy link

alurin commented Dec 13, 2013

New transaction decorator atomic (used in django admin) is not covered, e.g saves in admin is not sended existing monkey pathing signals.

And after reading https://code.djangoproject.com/ticket/21004: transaction.is_managed() in Django 1.6 always returns False.

I'm think because of this django-celery-transactions is fully broken. All post transaction tasks call method apply_async from super class.

Tested on Python 2.7(Ubuntu 10.10) with this list of packages:

django-celery==3.0.23
django-celery-transactions==0.1.3
git+git://github.com/toastdriven/[email protected]
git+git://github.com/notanumber/xapian-haystack.git
django-haystack-panel==0.1.3
celery-haystack==0.7.2

@nicpottier
Copy link

Not sure I would necessarily recommend it, but I just updated my fork to work with 1.6 (and 1.6 only). Sorry, can't be bothered hacking around everything to make it backwards compatible seeing as everything in 1.6 dealing with transactions is brand new.

But it seems to work, so hey, maybe it will be useful.

https://github.com/nyaruka/django-celery-transactions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants