-
Notifications
You must be signed in to change notification settings - Fork 79
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
fix issue where concat was used incorrectly #16
Conversation
It would be great if we can merge this and publish. Thanks! |
@eddiezane Wondering if you (or anyone else with admin) can do a quick code-review and merge? Thanks! |
Thanks for the ping, @limianwang! Apologies but I don't work for SendGrid any more. @thinkingserious or @mbernier might be able to help out though. |
Thanks! @thinkingserious @mbernier Can one of you review this? I'm kind of depending on this module. |
Hello @limianwang, Thanks for the pull request and apologies for the delay! Please submit a signed CLA so we can get this merged: https://github.com/sendgrid/smtpapi-nodejs/blob/master/CONTRIBUTING.md#cla Thank you for your support! With Best Regards, Elmer |
Hello @limianwang, Just checking in :) |
@thinkingserious ah sorry for the delay, somehow didn't see this notifications. Is that supposed to be email version? |
Yes, please email the signed CLA to [email protected]. Thanks! |
@thinkingserious Done. |
Hello @limianwang, |
No description provided.