Shift Bots event logic out of slackclient

[luke-john]

ie

events.js

import client from 'lib/client';
import message from 'message';

client.on('message', message);

message.js

import client from 'lib/client';

export default (message) => {
  const channel = client.getChannelGroupOrDMByID(message.channel);
  // do stuff
}

[kevincharm]

Yeah it would be good to modularise this. Perhaps still call the slack-client instance slack or maybe slackClient to be less ambiguous?

[luke-john]

Yep, client would not be a useful name.

Is there going to be any other interface with slack? If not just slack seems fine.

[karlbright]

This will be fixed by #23

[karlbright]

Fixed by Slack Application changes.