From 5a1753a7b06f5acf724958b872eec1ff794a2a59 Mon Sep 17 00:00:00 2001
From: Ankur Oberoi <aoberoi@gmail.com>
Date: Mon, 21 Aug 2017 16:09:43 -0700
Subject: [PATCH] fix/add comments

---
 src/bot.coffee    | 4 +++-
 src/client.coffee | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/bot.coffee b/src/bot.coffee
index 24acebc2..f99096b1 100644
--- a/src/bot.coffee
+++ b/src/bot.coffee
@@ -37,7 +37,9 @@ class SlackBot extends Adapter
     @client = new SlackClient(@options, @robot)
 
   ###
-  Slackbot reload users timeout (reloads every 1 hour)
+  Slackbot loads full user list on the first brain load
+  QUESTION: why do brain adapters trigger a brain 'loaded' event each time a key
+  is set?
   ###
   setIsLoaded: (@isLoaded) ->
 
diff --git a/src/client.coffee b/src/client.coffee
index 729354fc..86611c46 100644
--- a/src/client.coffee
+++ b/src/client.coffee
@@ -31,6 +31,7 @@ class SlackClient
   Open connection to the Slack RTM API
   ###
   connect: ->
+    # QUESTION: why do we throw away the login data?
     @rtm.login()