-
Notifications
You must be signed in to change notification settings - Fork 435
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
Digest Email Support #23
Comments
An important part of this would be to first create a batch concept more generally. For instance, if an event is triggered 1,000 times, there should be a method to bundle that into one notice. That notice would then form the basis of your digest email. |
I'm probably going to do a patch on engine.py around line 47 or so to allow for a notice_type/user grouping feature when emit_notices is signaled. |
A comment from a committer would be helpful here. |
Any more thoughts on this? I'm consider doing a fork two implement. Our users really want a digest mode. |
A digest mode is critical but after reading alot of this code I realized that django-notification and django-mailer are tightly coupled and they need to be worked on as a unit. In addition, the Pinax team wants to do a pluggable backend for the notification system - but I haven't seen any of that code. I'm sorry but I don't have any good comments right now - I haven't looked at the code in about 3 months and I remember noting that this would be a serious change. Cheers, |
Tristan, have you started the fork. Coincidentally, I now need this functionality. |
Hi AdamN - we are trying to work out the best way to implement internally. |
This might be best accomplished in jtauber/django-mailer |
Ok, I started this post on pinax-users: http://groups.google.com/group/pinax-users/browse_thread/thread/a493a88f5e9527c7 I don't think django-mailer ever implemented the digest functionality and it doesn't seem like the right place to have it anyway. I'm going to try to work on this today and tomorrow. |
Would be a good feature for email notifications, allowing a user to have all notifications added to one email to receive once per day.
The text was updated successfully, but these errors were encountered: