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

channelHistory example #47

Closed
timrabbetts opened this issue Aug 17, 2019 · 2 comments
Closed

channelHistory example #47

timrabbetts opened this issue Aug 17, 2019 · 2 comments

Comments

@timrabbetts
Copy link

I've connected all working but when I try and query channel 200 object but no messages?

$slack_token = "bla";
$slack_uid = "blabla";

$client = ClientFactory::create($slack_token);

$results = $client->channelsHistory(['count' => 100, 'token' => $slack_token, 'channel' => 'Cblabla']);
$messages = $results->getMessages();

print_r($messages);

@pyrech
Copy link
Member

pyrech commented Oct 9, 2019

Hello @timrabbetts, sorry for the delay. I didn't see your issue until now.

If you received no messages, I guess something is wrong for the Slack API. When there is a an error, Slack always return a 200 response, but with a property ok set to false and an error message.

With previous version (< 2.2) of this package, you needed to check the boolean property ok of the object to check if Slack really returned some data:

$results = $client->channelsHistory(['count' => 100, 'channel' => 'Cblabla']);

if (!$results->getOk()) {
    // Slack returned an error
}
$messages = $results->getMessages();

With the version >= 2.2, everything is now more simple because our SDK now throws an exception as soon as Slack return an error code.

That beiing said, in most cases, there is a problem with your token. Is it valid? Does it have the required scope to retrieve messages?

On a side note, you do not need to pass the token at each endpoint if you already provided it in the ClientFactory 😉

@pyrech
Copy link
Member

pyrech commented Oct 14, 2019

Closing as an explanation has been given. Feel free to reopen if you need more details. Thanks 😉

@pyrech pyrech closed this as completed Oct 14, 2019
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

No branches or pull requests

2 participants