Notify a slack channel as well as each user for all review hooks in rbslack

Review Request #8777 — Created Feb. 23, 2017 and discarded — Latest diff uploaded

Information

rb-extension-pack
master

Reviewers

This change changes the behaviour of notification to also send messages directly to users. This requires users in Slack and ReviewBoard to match (documentation added). This provides a nice feed for each user to get updates on only the reviews they care about. The users it notifies are the submitter, any people/group members assigned the to review and any commenters.

Also this adds a header on the payload sent, Content-type: application/json to support the Slack open-source alternative MatterMost. (This follows the slack documentation's request to have this header present)

Tested with MatterMost, which claims 100% api compatibility with Slack.

    Loading...