search

springload / madewithwagtail

A showcase of sites and apps made with Wagtail CMS, the easy to use, open source Django content management system
http://madewithwagtail.org
MIT License
85 stars 21 forks source link

Gracefully handle failing slack hook #113

Closed loicteixeira closed 6 years ago

loicteixeira commented 6 years ago

The Springload Slack hook was deleted (the Wagtail one is still operational), which would cause it to raise HTTPError (see Sentry MADEWITHWAGTAIL-9).

I removed the hook from local.py on the server (which will take effect on the next restart, probably when we merge and deploy this) but this PR should handle it more gracefully if it was to happen again.

loicteixeira commented 6 years ago

That's a fair point. I've added it to the agenda for the next meeting.

I will merge and deploy this for now.