CodeForPhilly / CodeForPhilly/clean-and-green-philly

Task: Refactor slack notification service

Open
#1,199 0 comments 0 reactions 1 assignee Claimed by @cfreedman View on GitHub
backend python
Dominant language
Python
Stars
50
Forks
92
PR merge metrics
No merged PRs in 30d

Description

Make slack notifications handled through a single `SlackNotifier` class that handles establishing the client connection and sending any messaging for pipeline run failures or warnings from data-diffing etc.

Currently the are several places across the repo that the web client connection is created and messages and being sent, so this should be centralized to one object.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.