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

Helpers: Personalization - From #1067

Open
lopenchi opened this issue Nov 22, 2021 · 3 comments
Open

Helpers: Personalization - From #1067

lopenchi opened this issue Nov 22, 2021 · 3 comments
Labels
status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap

Comments

@lopenchi
Copy link

lopenchi commented Nov 22, 2021

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:

public function addFrom($email)

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

@JenniferMah JenniferMah added status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap labels Dec 6, 2021
@JenniferMah
Copy link
Contributor

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.

@lopenchi
Copy link
Author

lopenchi commented Apr 1, 2022

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

@childish-sambino
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap
Projects
None yet
Development

No branches or pull requests

3 participants