Skip to content

Conversation

@foens
Copy link
Contributor

@foens foens commented Nov 21, 2013

Before only File where supported, however, Google App Engine does not
allow applications access to the filesystem, thus where unable to attach
any attachments.
This pull request fixes #8

Before only File where supported, however, Google App Engine does not
allow applications access to the filesystem, thus where unable to attach
any attachments.
@motdotla
Copy link
Contributor

This looks quite good. Thanks for this. I'm reviewing and then merging in.

motdotla added a commit that referenced this pull request Nov 21, 2013
Add support for adding attachments directly from an InputStream.
@motdotla motdotla merged commit ebbfcba into sendgrid:master Nov 21, 2013
@motdotla
Copy link
Contributor

Ok, this is live in 0.1.2. Also there is a README example.

https://github.com/sendgrid/sendgrid-java#attachments

@motdotla
Copy link
Contributor

@foens shoot me an email at motte@sendgrid.com. I'd like to mail you a shirt for your help with this.

@foens foens deleted the io-stream-attachments branch November 22, 2013 07:19
@foens
Copy link
Contributor Author

foens commented Nov 22, 2013

Please check pull #14. It updates the README to reflect the changes in the API. Will send you an email.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add attachment with contents from String.

2 participants