Skip to content

Conversation

@notheotherben
Copy link
Contributor

This should address the issues faced in #353 while still ensuring that we accurately cover the type's requirements.

Please let me know if you have any concerns or further requests for this fix.

Regards,
Benjamin

/cc @sheyaweidberg

This should address the issues faced in sendgrid#353 while still ensuring that we accurately cover the type's requirements.
@thinkingserious
Copy link
Contributor

Hello @spartan563,

Thank you for the PR!

We will be reviewing this PR soon, in the mean time, if you have not, please be sure to sign our CLA so that we can merge your changes. Thanks again!

Team DX

@thinkingserious thinkingserious added status: code review request requesting a community code review or review from Twilio status: cla needed and removed status: cla needed labels Jan 25, 2017
@jpinkster
Copy link

ETA on merge?

@thinkingserious
Copy link
Contributor

Hi @jpinkster,

I don't have a good ETA just yet, but I've added your vote to the issue to help it rise in our backlog.

Thanks!

@thinkingserious thinkingserious merged commit 24cd4bf into sendgrid:master Mar 14, 2017
thinkingserious added a commit that referenced this pull request Mar 14, 2017
@jpinkster
Copy link

🤘🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: code review request requesting a community code review or review from Twilio

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants