Username does not default to Wercker

[drewfranklin]

In the slack notification the username does not default to Wercker. It looks like environment variable WERCKER_PRETTY_SLACK_NOTIFY_USERNAME is not being defined in script/run, but I am not exactly sure if that is the issue

[spesnova]

@drewfranklin Thank you for reporting. Could you show me your wercker.yml ? I might be able to help you.

[drewfranklin]

For sure box: zdfs/docker-suit build: steps:

• npm-install
• script: name: create empty 'public' and 'deploy' directories code: |- mkdir 'public' mkdir 'deploy'`
• grunt: tasks: predeploy
• grunt: tasks: deploy
• script: name: copy documentation for deployment code: |- cp --recursive deploy/* "$WERCKER_OUTPUT_DIR"
• script: name: echo nodejs information code: | echo "node version $(node -v) running" echo "npm version $(npm -v) running" after-steps:
• wantedly/pretty-slack-notify: webhook_url: $SLACK_WEBHOOK_URL channel: suit notify_on: "failed" deploy: steps:
• duleorlovic/ftp-deploy: destination: ftp://$HOSTNAME/$FTP_TARGET username: $FTP_USERNAME password: $FTP_PASSWORD after-steps:
• wantedly/pretty-slack-notify: webhook_url: $SLACK_WEBHOOK_URL channel: suit deploy-release: steps:
• duleorlovic/ftp-deploy: destination: ftp://$HOSTNAME/$WERCKER_GIT_BRANCH username: $FTP_USERNAME password: $FTP_PASSWORD after-steps:
• wantedly/pretty-slack-notify: webhook_url: $SLACK_WEBHOOK_URL channel: suit

Sorry about the formatting but using the code markup makes it one line

[spesnova]

@drewfranklin Thank you for sharing it to me.

At first, you might be able to work around by specifying username option like this:

        - wantedly/pretty-slack-notify:
            webhook_url: $SLACK_WEBHOOK_URL
            channel: suit
            username: Wercker

To find the root cause, could you show me "wantedly/pretty-slack-notify" step's log on wercker?

[drewfranklin]

Awesome thanks for the quick fix Here is my log

[spesnova]

@drewfranklin Thanks! I could find the root cause from your log ❗️ This bug will be fixed in next release 😃
So please use username option to workaround until next version released.

[dtan4]

@drewfranklin step-pretty-slack-notify v0.3.4 including the fix of this issue was just released. Please try it.

[drewfranklin]

Yep it worked thanks so much everyone!