Define django_loaded signal so we can properly connect our own.
Review Request #968 — Created Aug. 16, 2009 and submitted — Latest diff uploaded
Email notification wasn't getting properly connected to the publish signals because the file wasn't loaded. And connecting them in __init__.py breaks because django isn't loaded yet. With the django_loaded signal, we can delay connecting signals until we're sure all we need is there.
All tests pass.
Diff Revision 1
This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.
orig
1
2
/trunk/reviewboard/signals.py |
---|
/trunk/reviewboard/urls.py |
---|
/trunk/reviewboard/notifications/__init__.py |
---|
/trunk/reviewboard/notifications/email.py |
---|