-
Notifications
You must be signed in to change notification settings - Fork 625
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
Fixing issue #686 #705
Fixing issue #686 #705
Conversation
test/unit/MailHelperTest.php
Outdated
@@ -70,4 +70,64 @@ public function testEmailName() | |||
'{"name":"\\"O\'Keeffe, John \\\\\\"Billy\\\\\\"\\"","email":"test@example.com"}' | |||
); | |||
} | |||
|
|||
public function testA() |
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.
The testA
method name is not proper because I don't know this test method actually tests.
Can you name the proper test method name?
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.
@peter279k done!
@peter279k done! |
@peter279k could you please approve? Its missing 2 points to win the swag |
Hi @reisraff, I've approved this, but it seems that |
1a9c6cc
to
8f29fba
Compare
@peter279k done! |
Hello @reisraff, |
Hello @reisraff, |
Could you please issue a release with this fix? It has been half a year and this patch, as far as I understand, is blocking normal usage of Personalization objects. Practically right now we have to construct and end custom json blobs to the API instead of actually using the library if we want to send multiple personalization objects in a proper way. Without it, sending wrong emails or an email with multiple people in the To-header is a little too easy. |
Fixes #686