search

fedora-infra / fas

Fedora Account System
https://admin.fedoraproject.org/accounts
GNU General Public License v2.0
40 stars 50 forks source link

Fix two separate fatals #121

Closed relrod closed 9 years ago

laxathom commented 9 years ago

ya, ya, :+1: on catching such error. though we have email delivery in few places which would need to be extracted and put at one place with a listener avoiding to fix this everywhere. I'm (re)thinking the whole notification system to handle more than one backend (email, fedmsg, etc). Will add that changes in fas/subscribers in the mean time.

pypingou commented 9 years ago

@laxathom so should this be merged or close?

laxathom commented 9 years ago

Closed.

relrod commented 9 years ago

@laxathom Should we at least cherry-pick 6e8939ec8a9fa1e34d32414bce74e1f825098695 from this?

laxathom commented 9 years ago

Sure thing.