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

MultiWebSocket #54

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

MultiWebSocket #54

wants to merge 13 commits into from

Conversation

CarlosPacheco
Copy link

Change: objects class move to the Models folder,
List will all the slackChannel(Group, DM, Channels).
Father class SlackChannel from all channels(Group, DM, Channels).
Fix bug when the socket receive a message with type null.
Fix the error when the slackSocketClient try open socket with
LoginReponse is not Ok,
Add more events, and now the event pass back the
slackclient/slacksocketclient

Change: objects class move to the Models folder,
List will all the slackChannel(Group, DM, Channels).
Father class SlackChannel from all channels(Group, DM, Channels).
Fix bug when the socket receive a message with type null.
Fix the error when the slackSocketClient try open socket with
LoginReponse is not Ok,
Add more events, and now the event pass back the
slackclient/slacksocketclient
@CarlosPacheco
Copy link
Author

Yes. i only add more things. btw i can add more examples to the documentation.

@Inumedia
Copy link
Owner

Hi CarlosPacheco,

Your branch has conflicts currently from the pull requests I merged in.

@Inumedia
Copy link
Owner

Inumedia commented Jul 1, 2016

@CarlosPacheco Your branch still has conflicts in it and can not be merged until they are resolved.

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