flake8
passed.
JSHint
passed.
Review Request #11211 — Created Oct. 8, 2020 and submitted
Currently, when reviewers select the "Ship It!" option, the notification sent
to Slack has a bar that is colored grey. I found that a value of 'success'
is passed to color. Upon investigation it seems that Slack and Mattermost
doesn't understand what color corresponds to 'success'. However, it knows
'good' corresponds to green.Evidence that Mattermost uses ‘good’ and not ‘success’:
https://mattermost.atlassian.net/browse/MM-3181
https://mattermost.com/blog/getting-started-with-mattermost-integrations/Evidence that Slack uses ‘good’:
https://api.slack.com/reference/messaging/attachments#fields
logo.png
at /var/folders/...
. See #integrationsSummary | ID | Author |
---|---|---|
0fb76967962a35ef4e395bb595404e5e3fa29bb9 | HailanXyouknow |