presence_change events being blocked

[magroski]

I was testing multiple events using your code and I noticed that this block of code

} else if (data.type === 'presence_change') {
   // update slackData presence when user becomes active/inactive
   for (var i in self.slackData.users) {
      if (self.slackData.users[i].id === data.user) {
         self.slackData.users[i].presence = data.presence;
         break;
      }
   }
}

is blocking the bot from processing presence_change events.

Adding cb(null, messageData); after the for loop easily solve this problem.

https://github.com/xBytez/slackbotapi/blob/master/lib/rtm.js#L240

[franciskim]

looking back that seems like the sensible thing to do, thanks for bringing that up! @magroski

[magroski]

Don't forget to push the changes to npm :)

[xBytez]

I wont! I'm still working on a bit of documentation and I'll then update npm with the new version!