Add support for attachments

cloudfoundry-community / slack-notification-resource

Concourse CI resource for sending notifications to Slack.

MIT License

75 stars 82 forks source link

Add support for attachments #23

Closed tshak closed 7 years ago

tshak commented 7 years ago

This adds support for Slack message attachments. Because it is both legal and common to post a message to Slack with attachments and no text, the text parameter is now handled in the following fashion:

When attachments IS specified, text and text_file are optional.
When attachments IS NOT specified, the old behavior of requiring text or text_file is retained.

As such, there should be no breaking changes. I updated the documentation to reflect this change.

analytically commented 7 years ago

Would be great to have this!

lnguyen commented 7 years ago

hey @tshak do you mind rebasing? and I'll merge it in?