search

apache / openwhisk-catalog

Curated catalog of Apache OpenWhisk packages to interface with event producers and consumers
https://openwhisk.apache.org/
Apache License 2.0
33 stars 49 forks source link

slack post action have channel parameter as required it should be optional #235

Closed csantanapr closed 6 years ago

csantanapr commented 7 years ago

I can configure a webhook and specify a default channel for the message. This allows the flexibility to change the channel in the web hook config, and also only required to bind one parameter url

we should remove the check if channel is passed https://github.com/apache/incubator-openwhisk-catalog/blob/master/packages/slack/post.js#L89

Also update the docs stating that minimum required param is url https://github.com/apache/incubator-openwhisk-catalog/tree/master/packages/slack#posting-a-message-to-a-slack-channel