Change slack format to attachment style #3852

Merged
merged 3 commits into from Jul 27, 2016

Projects

None yet

4 participants

@pblasquez
Contributor

As documented here: https://api.slack.com/docs/message-attachments

This change also makes things compatible with simple HTML templates by using strip_tags() on the template output.

@pblasquez pblasquez Change slack format to attachment style
As documented here: https://api.slack.com/docs/message-attachments

This change also makes things compatible with simple HTML templates by using strip_tags() on the template output.
cf94323
@pblasquez pblasquez formatting error
c0a0fab
@Rosiak
Contributor
Rosiak commented Jul 21, 2016

Uh, this seems interesting!

@Rosiak
Contributor
Rosiak commented Jul 21, 2016 edited

Tested, working good with my slack transport 👍
Perhaps we should add some lines to our documentation that these templates should now be possible to use here.

@laf
Member
laf commented Jul 23, 2016

@pblasquez Can you update the docs: http://docs.librenms.org/Extensions/Alerting/#transports-slack to add some info about this please.

@laf
Member
laf commented Jul 26, 2016

@pblasquez bump

@pblasquez pblasquez Adding new slack information
a5383b9
@laf laf added Documentation and removed Blocker labels Jul 27, 2016
@laf laf merged commit db00e26 into librenms:master Jul 27, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@pblasquez pblasquez deleted the pblasquez:patch-7 branch Aug 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment