• 
      
    Fish Trophy

    hailan got a fish trophy!

    Fish Trophy

    Change 'Ship It!' color 'success' to 'good' for chat integrations

    Review Request #11211 — Created Oct. 8, 2020 and submitted

    Information

    rbintegrations
    master

    Reviewers

    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

    • I ran the unit tests on Slack and Mattermost. Two unrelated tests are failing
      because they cannot find logo.png at /var/folders/.... See #integrations
    • Confirmed that "Ship It!" message has a green bar instead of a grey one in
      Slack and Mattermost for Review Board version 3.0 and 4.0 after the change
    Summary ID Author
    Change color 'success' to 'good'
    0fb76967962a35ef4e395bb595404e5e3fa29bb9 HailanXyouknow
    hailan
    MarcusBoay
    1. Ship It!
    2. 
        
    david
    1. Ship It!
    2. 
        
    hailan
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-1.0.x (2a41634)