Skip to content
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

Add/slack alert #970

Merged
merged 6 commits into from Apr 19, 2018

Conversation

Projects
None yet
3 participants
@DavidCramer
Copy link
Member

commented Apr 6, 2018

Adds Slack alert: props @robbiet480

robbiet480 and others added some commits Mar 5, 2018

Merge pull request #962 from Bloomjoy/slack-alert
Add Slack alert implementation
@robbiet480

This comment has been minimized.

Copy link
Contributor

commented Apr 6, 2018

Nice!

@DavidCramer DavidCramer changed the title [WIP] Add/slack alert Add/slack alert Apr 6, 2018

@DavidCramer DavidCramer requested a review from lukecarbis Apr 6, 2018

@DavidCramer

This comment has been minimized.

Copy link
Member Author

commented Apr 6, 2018

@lukecarbis This is all cleaned up, tested and ready for your review.

@DavidCramer DavidCramer changed the base branch from develop to add/real-client-ip Apr 6, 2018

@DavidCramer DavidCramer changed the base branch from add/real-client-ip to develop Apr 6, 2018

@lukecarbis lukecarbis merged commit 639f3c1 into develop Apr 19, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@lukecarbis lukecarbis deleted the add/slack-alert branch Apr 19, 2018

@robbiet480

This comment has been minimized.

Copy link
Contributor

commented Apr 19, 2018

Thanks @lukecarbis!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.