codeforamerica / intake

A Django project behind the Clear My Record website
https://clearmyrecord.codeforamerica.org
MIT License
49 stars 23 forks source link

Fix unread apps notification in staging #1384

Closed hartsick closed 4 years ago

hartsick commented 4 years ago

Previously if an organization didn't have an email address for a profile for some reason (was happening on staging for some counties we hadnt' yet onboarded), the job to send unread application notifications for the day would bail out early, preventing some orgs from receiving notifications.

This commit attempts to fix that by catching the raised exception, collecting it in case there are multiple, and re-raising it at the end of the task.