-
Notifications
You must be signed in to change notification settings - Fork 92
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
ooaklee/Add-mattermost-notifier-integration #119
base: master
Are you sure you want to change the base?
Conversation
- Added logic to config to map mattermost notifier - Added packages needed to send a message with mattermost notifier - Added client for interacting with mattermost - Added mattermost notifier case - Added mattermost notifier to README
- Updated env vars name (slack > mattermost)
- Enabled Markdown in payload - Added screenshot for Mattermost message
…the ChatPostMessage errors as we expect
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for your contribution 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually sorry one thing, can you resolve the conflicts? it's just dependencies
Hey @drlau, sorry about the delay. The conflicts have been resolved 😊 |
WHAT
Mattermost
as a notifierWHY
Many people/ small teams are migrating over to
Mattermost
with the recent changes toSlack
. With this update, they'll be able to migrate this awesome tool with them and continue to receive their updates.