Added support for Facebook "attachment" type of objects for conversat…

[tkornblit]

…ions.

[crummy]

Works great for me, thanks - just what I was looking for.

BTW, I had to add lodash to package.json to build it.

Here's how I tested it:

  var sayRandomImage = function(convo, urls) {
    var imageUrl = urls[Math.floor(Math.random()*urls.length)];
    var msg = {
      "attachment": {
        "type": "image",
        "payload": {
          "url": imageUrl
        }
      }
    };
    convo.say(msg);
  }

[tkornblit]

Cool brother. What are you working on?

Tom Kornblit

On May 10, 2016, at 1:53 PM, Malcolm Crum notifications@github.com wrote:

Works great for me, thanks - just what I was looking for.

BTW, I had to add lodash to package.json to build it.

—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub #208 (comment)

[crummy]

Experimenting with an interactive resume bot for my girlfriend. Images were a must :)

[crummy]

One thing I noticed is that image messages sometimes arrive out of order with text messages. Not sure why, perhaps that's a facebook issue. I'm considering increasing the delay between messages to help.

[tkornblit]

Use botkits Conversation class. It has a built-in queue mechanism.

Tom Kornblit

On May 10, 2016, at 3:32 PM, Malcolm Crum notifications@github.com wrote:

One thing I noticed is that image messages sometimes arrive out of order with text messages. Not sure why, perhaps that's a facebook issue. I'm considering increasing the delay between messages to help.

—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub #208 (comment)

[benbrown]

@tkornblit thanks for the PR. I'm merging in the attachment related features here, but not sure about the changes you've made to the findConvo and others. Maybe open a new PR if you want to get that stuff included?