sendgrid-php icon indicating copy to clipboard operation
sendgrid-php copied to clipboard

Helpers: Personalization - From

Open lopenchi opened this issue 4 years ago • 3 comments

Issue Summary

Checking the request body for mail/send (https://docs.sendgrid.com/api-reference/mail-send/mail-send), it was discovered that helper classes provided in this library can be improved

Errors found:

  • In Presonalization class there is a property $from, which is an object. Based in all the code the object setter method is setObjectName, but in this case is addFrom as the from property is an array, but it is an object

Code: https://github.com/sendgrid/sendgrid-php/blob/3dcfc4db6583d5fa581248bb486418963804620d/lib/mail/Personalization.php#L69

This is also confusing because in Mail class the method is setFrom

lopenchi avatar Nov 22 '21 17:11 lopenchi

Hi @lopenchi! Thanks for bringing this to our attention. This issue has been added to our internal backlog to be prioritized. Pull requests and +1s on the issue summary will help it move up the backlog.

JenniferMah avatar Dec 06 '21 23:12 JenniferMah

@JenniferMah Can you send me the link for the ticket ? I would like to +1

lopenchi avatar Apr 01 '22 16:04 lopenchi

@lopenchi Putting a 👍 on the issue summary is what's needed.

childish-sambino avatar Apr 05 '22 15:04 childish-sambino