search

datamade / django-councilmatic-notifications

Notifications app for Django Councilmatic
MIT License
1 stars 0 forks source link

Narrow the notifications email. #20

Open reginafcompton opened 7 years ago

reginafcompton commented 7 years ago

The notifications email contains too many notifications. Let's limit alerts to the following:

Bills The bill has a new action added to its history.

Committees The committee schedules a new meeting, has new legislative activity, or adds a new member, but not for trivial things, e.g. a new description.

Person The person sponsors new legislation or joins a committee, but not for trivial things, e.g. a new email.

Events The event changes (e.g., new time, date, document).

@evz Is such narrowing possible? Looking at the SQL, I think it might get messy.....

reginafcompton commented 7 years ago

Also....if we can make these modifications, then we should also change the notifications email to include more explanatory text about what's new and alert-worthy.

evz commented 7 years ago

So, it seems like it also only makes sense to send notifications for events if the events are in the future. Is that the case?