MuckRock / muckrock

MuckRock's source code - Please report bugs, issues and feature requests to info@muckrock.com
https://www.muckrock.com
GNU Affero General Public License v3.0
114 stars 22 forks source link

Notification emails? #157

Closed shawnmusgrave closed 9 years ago

shawnmusgrave commented 9 years ago

I didn't get a daily notification email today, even though I had a request updated within the past 24 hours:

https://www.muckrock.com/foi/united-states-of-america-10/odni-civil-liberties-and-privacy-guidance-for-ic-10430/

The request is embargoed, but I received no notification for this or any other request of mine updated in the 24 hours.

I'm worried this might have impacted other users.

allanlasser commented 9 years ago

Closing because unreproducible.

shawnmusgrave commented 9 years ago

Did we confirm that notifications went to other users?

allanlasser commented 9 years ago

Yes, this isn't a problem with the update emails but with your request not getting updated in particular. It's a problem I'm keeping my eyes open for but can't replicate now.

morisy commented 9 years ago

So as a point of reference, on a typical day about 16 summary emails go out. On Monday a.m., zero summary emails went out. On Tuesday, 2 went out — both of which only had an update for "Awaiting Response."

I think this was an error that occurred at the same time as these two messages being sent out today:

https://app.getsentry.com/app4858795herokucom/app4858795herokucom/group/45902141/

I would place a high probability that something is breaking these messages, and that error is the key to bringing this injustice to light.

allanlasser commented 9 years ago

See the above message, I fixed the bug referenced by Michael's Sentry link. These messages should start getting sent out again, keep an eye open.

morisy commented 9 years ago

This appears all fixed. Thanks! :+1: