-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slack For Incident Notifications #212
Comments
Yes @michaeljguarino, we can start with the public slack app. We can add authentication to it as well. We can trigger the slack webhook from n8n as well but I have a few questions -
|
The webhook system does persist into the database and retry so that should help with that. My thought was there should be webhook types for standard formats like slack, teams, etc and we add logic to change the way the event is delivered there from simply serializing the object to for instance generating a slack message using their block system |
Yes @michaeljguarino, slack messages using blockit builder (blocks system) and similarly for teams (using adaptive cards designer) messages are doable.
|
With slack, you can set the webhook endpoint to an incoming webhook which doesn't need a secret, so that seems doable, we might need to store auth secrets for others though. Ideally we make the big destinations like slack as first class citizens, then have generic webhooks for the rest of the usecase. The eligible events are implemented here: https://github.com/pluralsh/plural/blob/master/apps/core/lib/core/pubsub/protocols/deliverable.ex and of course we can add others. We'll need to improve the payload function to support the various webhook types basically |
Use Case
We currently have a generic webhook systems incidents send to, which can be picked up by n8n or zapier for further integrations, but we should add first-order support for slack (and perhaps mattermost/teams?) since it'll be so useful
Ideas of Implementation
Message from the maintainers:
Excited about this feature? Give it a 👍. We factor engagement into prioritization.
The text was updated successfully, but these errors were encountered: