slack-github-action icon indicating copy to clipboard operation
slack-github-action copied to clipboard

Action does not error when message could not be sent

Open gr2m opened this issue 4 years ago • 1 comments

Description

I'm new to Slack's APIs, and I only installed the action. I missed to install the new app to the channel I want it to send a message to, the PR at #5 would have saved me some time ;)

I wanted to let you know that although the error is logged to the output, the action still exists as if sending the message succeeded.

What type of issue is this? (place an x in one of the [ ])

  • [x] bug
  • [ ] enhancement (feature request)
  • [ ] question
  • [ ] documentation related
  • [ ] example code related
  • [ ] testing related
  • [ ] discussion

Requirements (place an x in each of the [ ])

  • [x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
  • [x] I've read and agree to the Code of Conduct.
  • [x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report

Filling out the following details about bugs will help us solve your issue sooner.

Reproducible in:

package version: latest

node version: any

OS version(s): any

Steps to reproduce:

  1. Create new slack app
  2. Install on workspace
  3. Without adding the app to a channel, try to send a message to it

Expected result:

Action should fail with helpful instructions

Actual result:

Error is logged, but exist status is still 0

Attachments:

Sorry I've re-ran the action and have no longer access to the run that failed

gr2m avatar Aug 21 '21 21:08 gr2m

Thanks for the great input. Displaying actionable error messages would be more helpful to developers.

seratch avatar Nov 07 '21 22:11 seratch