-
Notifications
You must be signed in to change notification settings - Fork 839
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
api_call should update server state in rtm mode #74
Conversation
self.server.attach_channel(kwargs["user"], result["channel"]["id"], []) | ||
elif method in ('mpim.open', 'groups.create', 'groups.createchild'): | ||
if "ok" in result and result["ok"]: | ||
self.server.attach_channel(result['group']['name'], result['group']['id'], []) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both here and line 32 I think you should pass through the members
attribute rather than an empty array or passing a None
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was emulating the style of _client.py:SlackClient:process_changes, which also passes an empty. I'll update it, one moment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I'm not 100% familiar with the python client yet, so I think you should pass the members array, but that's not canon.
api_call should update server state in rtm mode
Ok, I'll look to push a 1.0.0 release to pypi later today. Thanks for contributing! |
@elementc, nice job! |
api_call should update server state in rtm mode
Fix for #73. Also closes #47 by way of solving the same problem that #70 does. I'm closing #70 myself.