-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Additional payload options #22
Comments
hey! I believe you can do this... doesn't https://github.com/clonn/slack-node-sdk/blob/master/src/lib/slack.seed.coffee#L77 set any options you pass through on the request? |
@kevinburkeshyp I think maybe that works with the API but not webhooks, because only certain keys are being added to the payload https://github.com/clonn/slack-node-sdk/blob/master/src/lib/slack.seed.coffee#L36 |
I agree with @farski. In my case, I want to override the |
+1 from me also, for now I've forked and added these options (master...simonrand:allow_unfurl_options) but would be nice to have this upstream.. (Not sure my changes are worth a PR? A little unclear how to add tests for these options..) |
There are some additional parameters that can be sent along with the payload (
link_names
,mrkdwn
, etc), that currently get stripped out because the payload is being built explicitly. Would it be possible to make it so those options could get passed along?The text was updated successfully, but these errors were encountered: