search

wantedly / step-pretty-slack-notify

Posts wercker build/deploy status to a Slack channel
MIT License
88 stars 34 forks source link

notify_on not working #21

Closed LennardWesterveld closed 8 years ago

LennardWesterveld commented 8 years ago

Hi there,

My build is successful and still getting slack notification. So the notify_on is not working, i think the environment name is wrong.

    after-steps:
        - wantedly/pretty-slack-notify@0.2.11:
            webhook_url: *******
            notify_on: "failed"
            channel: ******
            username: *****

Result from wercker.

export WERCKER_PRETTY_SLACK_NOTIFY_BRANCHES=""
export WERCKER_PRETTY_SLACK_NOTIFY_NOTIFY_ON="failed"
koudaiii commented 8 years ago

Thank you for feed back :+1: I will try :) Please wait a little. :bow:

dtan4 commented 8 years ago

We fixed this bug at #22 , and released as v0.2.12 :shipit: Please try this. Thank you for reporting the issue :exclamation:

LennardWesterveld commented 8 years ago

Hi there is still not working for me, the exported ENV is still WERCKER_PRETTY_SLACK_NOTIFY_NOTIFY_ON i see in code thats it needs to be WERCKER_PRETTY_SLACK_NOTIFY_ON

koudaiii commented 8 years ago

Sorry...Thank you for feed back. :bow: Released v0.2.13 :exclamation: :exclamation: