Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Twilio SMS outgoing media attachments (MMS) #951

Merged
merged 4 commits into from
Aug 29, 2017

Conversation

krismuniz
Copy link
Contributor

@krismuniz krismuniz commented Aug 10, 2017

Hello Botkit team 👋

PR Summary

This PR adds support for Twilio SMS outgoing media attachments (MMS) along with documentation on how use the feature.


Sending media attachments (MMS)

To send media attachments, pass a mediaUrl property to any of Botkit's outgoing messages functions (reply, say, ask, etc.) with an optional text property for text that goes along with your attachment.

/*
  Sending an attachment
*/
bot.reply(message, {
  text: 'Optional text to go with attachment',
  mediaUrl: 'https://i.imgur.com/9n3qoKx.png'
})

Note: your Twilio number as well as the recipient's phone must support MMS for media attachments to work

For more details on outgoing media attachments and a full list of accepted MIME types, go to Twilio's docs on media attachment.

Sending an attachment
*/
bot.reply(message, {
body: 'Optional body to go with text',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this field should be called text - it gets renamed during the send process

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I missed that. Fixed!

@benbrown benbrown merged commit 9500fb5 into howdyai:master Aug 29, 2017
@krismuniz krismuniz deleted the twilio-sms-media-att branch August 29, 2017 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants