search

jiaaro / django-alert

Send alerts, notifications, and messages based on events (i.e. signals) in your django application
https://github.com/jiaaro/django-alert
MIT License
60 stars 15 forks source link

Fix django17 warnings #8

Closed dulacp closed 9 years ago

dulacp commented 9 years ago

Some small fixes to resolve Django 1.7 warnings due to naïve date usage even if USE_TZ is set to True.

Cheers

dulacp commented 9 years ago

it fails due to the Django 1.4 support. Do you want to still support that version @jiaaro ?

jiaaro commented 9 years ago

yeah, unfortunately 1.4 is an LTS release and is still supported by the django foundation. I generally try to keep the same support schedule as the django project itself

dulacp commented 9 years ago

Alright, good to know. So, do you know a method to properly patch this, or do I have to make hasattr(manager, 'get_queryset') call to find out which method to use :/ ?

jiaaro commented 9 years ago

you could use self.all() instead maybe?

dulacp commented 9 years ago

I'll fix those warning later so, when Django project do not support them anymore :) Cheers