Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request enhance the plugin with partial support for image attachment.
To add image to sent sms the
attachments
property of theoptions
should contains an object which has for key the filename of the attached images and for value a base64 string representation of the image, example:IOS Support
The new feature works fine on IOS (at least based on my tests).
Android Support
The plugin only support the
INTENT
method for image attachement.Also providing a non empty phone number and attaching multiple image does not work.
I can't understand why but
Intent.ACTION_SEND
supports providing sms number whileIntent.ACTION_SEND_MULTIPLE
does not (Intent.ACTION_SEND_TO
does not support attachment ...).Windows
Not implemented