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

Adds an attachment builder that supports InputStream content. #160

Conversation

dmitraver
Copy link
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
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
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
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
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
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