Skip to content
This repository has been archived by the owner on Mar 4, 2021. It is now read-only.

SMS Media Urls #366

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

SMS Media Urls #366

wants to merge 2 commits into from

Conversation

jpwalters
Copy link
Contributor

This enhancement lays the groundwork for being able to add attachments to sms messages. Specifically I'm wanting to be able send sms/mms messages with file attachments from the SMS Conversation plugin and this enhancement would allow my plugin to do that without having to either duplicate code in OpenVBX or tell the user how to replace code files in OpenVBX and therefore making for a messy upgrade path for the user going forward.

SMS Conversation Plugin: Add Image Attachment Feature Request

Adding support to be able to optionally send attachments with a sms/mms message.
- If a blank mediaURL is passed in we need to convert it to a null or twilio will complain about the url being in the wrong formate.
jpwalters referenced this pull request in jpwalters/OpenVBX-Plugin-SMS-Conversation Jun 2, 2016
This is the beginning of being able to add a file or image attachment to a sms message when sending.

Issue #9
@boblaw
Copy link

boblaw commented Jun 2, 2016

Guys,

Would love to see MMS support added to openvbx.

@apips10
Copy link

apips10 commented Aug 24, 2016

I can second the request for MMS in OpenVBX.

@sethrock
Copy link

Yes, I raise my hand on that front too. Why isnt there mms support already? Would be kinda nice....

On Aug 24, 2016 at 3:06 PM, <apips10 (mailto:notifications@github.com)> wrote:

I can second the request for MMS in OpenVBX.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (#366 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/APNuNhLo7sPR_x6RNRFTzD3_s3SdjYNXks5qjMBLgaJpZM4IJa2v).

@greatteacherjosh
Copy link

+1 on this change.

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.

5 participants