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

Added optional callback to channel.postMessage #64

Closed
wants to merge 1 commit into from
Closed

Added optional callback to channel.postMessage #64

wants to merge 1 commit into from

Conversation

pveyes
Copy link

@pveyes pveyes commented Aug 7, 2015

channel.postMessage is missing optional callback because it ends up hitting _apiCall. I just realized this after having race condition of multiple postMessage call.

@ghost
Copy link

ghost commented Feb 13, 2016

Ok, first off, I apologize for how long it's taken us to address this PR.

I just pushed the 2.0.0 release for this library, which is a ground up refactor with the aim of building a lib that both Slack and our external contributors can easily work with and improve.

It also refactors the way that the web API is handled, providing individual function calls for each API endpoint. In the case of the chat.postMessage call it also adds in an optional callback.

@ghost ghost closed this Feb 13, 2016
aoberoi added a commit to aoberoi/node-slack-sdk that referenced this pull request Mar 20, 2019
This pull request was closed.
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