search

Irma-Response / irma-api

Irma Needs API, a fork of Harvey Needs
https://irma-api.herokuapp.com
GNU General Public License v3.0
9 stars 8 forks source link

add slack notification integration #19

Open roozbeh opened 7 years ago

roozbeh commented 7 years ago

To handle problem mentioned: https://github.com/Irma-Response/irma-api/issues/18

tdooner commented 7 years ago

I was chatting with @roozbeh about this in DM, but let me just put some thoughts here. I think this is a good idea, so that a slack channel can be used to remind us that updates are happening in the API.

But, I think it is going to be very noisy at our current velocity, which would mean that everyone would just leave the updates channel, thus undermining the feature.

Coupled with the fact that this is not tested at all nor does it contain any error handling, I'm going to say we should wait to merge this until after the storm passes.