Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Commit

Permalink
PR review changes
Browse files Browse the repository at this point in the history
  • Loading branch information
ryansydnor committed Oct 28, 2019
1 parent 2a4f25a commit 92dc086
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/CoreBot.js
Original file line number Diff line number Diff line change
Expand Up @@ -1585,7 +1585,7 @@ function Botkit(configuration) {
}));
botkit.webserver.use(express.static(static_dir));

botkit.webserver.server = botkit.webserver.listen(
botkit.httpserver = botkit.webserver.listen(
botkit.config.port,
botkit.config.hostname,
function() {
Expand Down
5 changes: 3 additions & 2 deletions lib/SlackBot.js
Original file line number Diff line number Diff line change
Expand Up @@ -566,9 +566,10 @@ function Slackbot(configuration) {
// normalize fields to match the rtm message format
message.user = message.user.id;

// home and modal surfaces don't have channel IDs
// home and modal surfaces don't have channel IDs -.-
// we default channel ID to user ID so that string operations will still work
if ((message.view && (message.view.type === 'home' || message.view.type === 'modal'))) {
message.channel = {};
message.channel = message.user.id;
} else {
message.channel = message.channel.id;
}
Expand Down

0 comments on commit 92dc086

Please sign in to comment.