Skip to content
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

Support for webhook url instead of company name and token. #6

Merged
merged 2 commits into from
Feb 11, 2015

Conversation

rafaelgaspar
Copy link
Contributor

Support for the new webhook URL form instead of company name and token.

@rafaelgaspar
Copy link
Contributor Author

Fixes #3

@esanchezm
Copy link
Owner

Thanks again. Could you please update the README file to document it? Actually, if this is the new method, feel free to remove the deprectated way.

@rafaelgaspar
Copy link
Contributor Author

The old way will still work for awhile as far as I know, so it's best to keep it backwards compatible until slack team removes this.

I'll update the README.

@rafaelgaspar
Copy link
Contributor Author

Rebased with current master and changed README.

@esanchezm
Copy link
Owner

Sorry, I didn't mean to remove the code, but the documentation regarding the old system, which is exactly what you did :)

esanchezm added a commit that referenced this pull request Feb 11, 2015
Support for webhook url instead of company name and token.
@esanchezm esanchezm merged commit 0101878 into esanchezm:master Feb 11, 2015
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.

2 participants