swaps out rtm message subtypes for discreet messages for enter and le…

…ave, fixes #296

src/bot.coffee

@@ -50,7 +50,7 @@ class SlackBot extends Adapter
       @client.loadUsers @usersLoaded
     else
       @isLoaded = true
-    
+
     # Brain will emit 'loaded' the first time it connects to its storage and then again each time a key is set
     @robot.brain.on "loaded", () =>
       if not @brainIsLoaded
@@ -239,16 +239,6 @@ class SlackBot extends Adapter
             @receive message
           )
 
-        # NOTE: channel_join should be replaced with a member_joined_channel event
-        when "channel_join", "group_join"
-          @robot.logger.debug "Received enter message for user: #{user.id}, joining: #{channel}"
-          @receive new EnterMessage user
-
-        # NOTE: channel_leave should be replaced with a member_left_channel event
-        when "channel_leave", "group_leave"
-          @robot.logger.debug "Received leave message for user: #{user.id}, leaving: #{channel}"
-          @receive new LeaveMessage user
-
         when "channel_topic", "group_topic"
           @robot.logger.debug "Received topic change message in conversation: #{channel}, new topic: #{event.topic}, set by: #{user.id}"
           @receive new TopicMessage user, event.topic, event.ts
@@ -260,6 +250,18 @@ class SlackBot extends Adapter
             @receive message
           )
 
+    else if event.type is "member_joined_channel"
+      # this event type always has a channel
+      user.room = channel
+      @robot.logger.debug "Received enter message for user: #{user.id}, joining: #{channel}"
+      @receive new EnterMessage user
+
+    else if event.type is "member_left_channel"
+      # this event type always has a channel
+      user.room = channel
+      @robot.logger.debug "Received leave message for user: #{user.id}, joining: #{channel}"
+      @receive new LeaveMessage user
+
     else if event.type is "reaction_added" or event.type is "reaction_removed"
 
       # Once again Hubot expects all user objects to have a room property that is used in the envelope for the message

src/client.coffee

@@ -50,6 +50,8 @@ class SlackClient
     @rtm.on "reaction_added", @eventWrapper, this
     @rtm.on "reaction_removed", @eventWrapper, this
     @rtm.on "presence_change", @eventWrapper, this
+    @rtm.on "member_joined_channel", @eventWrapper, this
+    @rtm.on "member_left_channel", @eventWrapper, this
     @rtm.on "user_change", @updateUserInBrain, this
     @eventHandler = undefined