Skip to content
This repository was archived by the owner on Dec 8, 2017. It is now read-only.

Conversation

mbland
Copy link

@mbland mbland commented Dec 15, 2015

Just sends them straight through without any special treatment.

cc: @ccostino @jeremiak

Mike Bland added 2 commits December 15, 2015 13:08
Just sends them straight through without any special treatment.
Since this package is intended to be a short-lived fork installed directly
from GitHub, it needs to have the CoffeeScript files pre-compiled.
@mbland mbland force-pushed the handle-reaction-added branch from 8f72fa4 to 770f730 Compare December 15, 2015 20:27
@mbland
Copy link
Author

mbland commented Dec 15, 2015

Note that the only files I really meant to include in this PR are src/client.coffee and test/client.coffee from c4a6cd1. I had to add 770f730 to get the direct-from-github install to work for 18F/hubot-slack.

afeld added a commit that referenced this pull request Dec 15, 2015
Handle reaction_added messages
@afeld afeld merged commit 16a5995 into master Dec 15, 2015
@afeld afeld deleted the handle-reaction-added branch December 15, 2015 21:24
mbland pushed a commit to 18F/hubot-adapter-slack-bot that referenced this pull request Jan 3, 2016
Converts them to SlackTextMessages before passing them to Hubot.

Squashed from PRs/commits:

  18F/node-slack-client#1:
    0b04a0b Handle reaction_added messages

  18F/node-slack-client#2:
    c65c172 Fix reaction_added message handling

  18F/node-slack-client#3:
    fd6fe4d Set user.room for reaction_added events
    129bbd6 Add clarifying comment to reactionAdded
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants