Skip to content

Add the option to use a proc to modify the data being sent by a webhook #233

Open
TikiTDO wants to merge 1 commit intomojombo:masterfrom
TikiTDO:master
Open

Add the option to use a proc to modify the data being sent by a webhook #233
TikiTDO wants to merge 1 commit intomojombo:masterfrom
TikiTDO:master

Conversation

@TikiTDO
Copy link

@TikiTDO TikiTDO commented Mar 29, 2016

I have a use case where I would like to post a notification to a webhook, but I want to control the data being posted. This implements a process_data argument on the Webhook handler. This argument can be set to a proc data, which receives all 5 of the notify arguments, and returns the object that will be posted/sent as json.

@TikiTDO
Copy link
Author

TikiTDO commented Mar 29, 2016

There appears to be an issue with the version of bundler that travis-ci uses, which I reported here. The actual code should pass all tests when these issues are sorted out.

@TikiTDO TikiTDO changed the title Add the option to use a modify the data being sent by a webhook with a proc object Add the option to use a proc to modify the data being sent by a webhook Mar 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant