rleger / covidrea

GNU Affero General Public License v3.0
3 stars 10 forks source link

Refactor RecordMailGunWebhook (job) #71

Open rleger opened 4 years ago

rleger commented 4 years ago

Awful thing I wrote very quickly. Refactoring would be most welcome.

One thing in particular from a practical standpoint I keep getting exceptions from old emails I openned after having cleared the DB of the corresponding prospect (or invite) record.. this is annoying. If the firstOrCreate fails we should log one line with the failure and not the whole exception (my slack is going to go crazy otherwise).

Thanks !