Skip to content

Adds an attachment builder that supports InputStream content.#160

Merged
thinkingserious merged 1 commit intosendgrid:masterfrom
dmitraver:read-attachments-from-inputstrem_134
Mar 22, 2017
Merged

Adds an attachment builder that supports InputStream content.#160
thinkingserious merged 1 commit intosendgrid:masterfrom
dmitraver:read-attachments-from-inputstrem_134

Conversation

@dmitraver
Copy link
Copy Markdown
Contributor

Fixes #134.

Adds an attachment builder class that helps creating content from InputStream as well as simple strings.

@thinkingserious thinkingserious added type: community enhancement feature request not on Twilio's roadmap status: cla signed status: work in progress Twilio or the community is in the process of implementing labels Oct 18, 2016
@dmitraver
Copy link
Copy Markdown
Contributor Author

Surprisingly the build has failed although I haven't changed any of the existing code except that adding an inner static class. I will check whats wrong with it.

@thinkingserious
Copy link
Copy Markdown
Contributor

thinkingserious commented Oct 19, 2016

Hi @dmitraver,

The tests will not work on Travis CI while this ticket is unresolved: #159

For now, please just make sure the tests pass locally with this procedure: https://github.com/sendgrid/sendgrid-java/blob/master/CONTRIBUTING.md#testing

Sorry for the inconvenience :(

@dmitraver
Copy link
Copy Markdown
Contributor Author

@thinkingserious

Yeah, I run all the tests locally and everything was fine.

@thinkingserious thinkingserious merged commit 6c96eea into sendgrid:master Mar 22, 2017
@thinkingserious
Copy link
Copy Markdown
Contributor

Thanks again for your PR @dmitraver!

Please take a moment to fill out this form so that we may send you some swag :)

thinkingserious added a commit that referenced this pull request Mar 22, 2017
@dmitraver
Copy link
Copy Markdown
Contributor Author

dmitraver commented Mar 23, 2017 via email

RichardCSantana-zz pushed a commit to RichardCSantana-zz/sendgrid-java that referenced this pull request Apr 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: work in progress Twilio or the community is in the process of implementing type: community enhancement feature request not on Twilio's roadmap

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants