-
Notifications
You must be signed in to change notification settings - Fork 22
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
Task 8 #13
Task 8 #13
Conversation
Thanks, let me find someone who can review this pull request |
@krzyk please review |
FYI: I merged changes from the jcabi-email master repository. |
@krzyk please take a look (rather annoying bug, would be great to fix it asap) |
@@ -70,6 +74,42 @@ public void cachesPreviousCall() throws Exception { | |||
} | |||
|
|||
/** | |||
* Test for issue #8 (unicode/UTF-8 is broken). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please write what the test does, not a link to issue
@krzyk Thanks for your feedback. I fixed the issues. Please look again. |
@@ -70,6 +73,43 @@ public void cachesPreviousCall() throws Exception { | |||
} | |||
|
|||
/** | |||
* Verifies that the encoding of the mail is set correctly and - as a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good description, but I think it is too long, let's go with: Envelope should handle non-Latin characters.
@rultor merge pls |
@rultor deploy now pls |
Issue URL: #8