Username does not default to Wercker

[Drew Franklin]

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

[Seigo Uchida]

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

[Drew Franklin]

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

[Seigo Uchida]

@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?

[Drew Franklin]

Awesome thanks for the quick fix
Here is my log

[Seigo Uchida]

@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.

[Daisuke Fujita]

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

[Drew Franklin]

Yep it worked thanks so much everyone!