-
Notifications
You must be signed in to change notification settings - Fork 780
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
API Simplification #378
API Simplification #378
Conversation
@thinkingserious I've created the above proposal for the 4 use cases required. Mostly this is adapted from my Additional observations:
I've already signed the CLA, but my Github username changed a while ago, so that might need to be updated in your system (used to be Let me know your thoughts and how you'd like to open this up for review by the community. Keen to start working on it once it's been approved. |
Hello @adamreisnz! I am in agreement with your design. I believe those interested in this redesign should follow along here. If it becomes difficult to communicate, we can explore other options. I have alerted people that may be interested in this re-design. |
With regards to the CLA, we rolled out a new, greatly simplified process. All you need to do now is fill out a short form and click a button. Please do that here. Thanks! |
Would be great if this was ironed out quickly. I know several startups here in Silicon Valley who have moved away from sendgrid because of the new API design :(. We are also thinking about it |
Thanks for the feedback @thebigredgeek. Do you have any thoughts on the new Mail Helper? |
The API looks simple to use. The question, to me, is how long we will need to wait to start using it. The new V3 "API" has been very painful for us over the past few months, and has cost us countless man hours trying to debug bizarre sendgrid issues that we never had to deal with before. |
Thanks for the additional feedback! Right now there are too many variables for me to give an accurate release date. This particular issue, is at a high priority and we will do all we can to support @adamreisnz to get this ready for a stable release. With regards to your current issues, please feel free to reach out to our team directly ([email protected]) or here on GitHub, we are happy to help. Also, it would be great to know the particular pain points you ran into so that we don't duplicate them with this next iteration. |
@SendGridDX Great! Sounds like you guys will hopefully ship soon. The issues with the old V3 "API" were caused by the horrendous design... "phase 1" or not... which required the gluing together of tens to hundreds of custom objects in order to send a single email. #335. It seems as though whoever was managing the transition to the V3 API didn't have much experience managing open source projects, as the V3 API shouldn't have even shipped if it wasn't finished and the legacy version should have been retained as the "production" lib until that time... Otherwise, you are literally screwing over the thousands of developers who are using this library to send email... along with your paying customers who are employing said developers. I am all for a better API, but you shouldn't roll out unfinished code to NPM as the "latest stable version". That's a very poor management decision in my opinion, since it probably cost you guys thousands of dollars in monthly revenue. |
Thank you @thebigredgeek once again for the valuable feedback. We will continue to make iterative improvements based on feedback such as yours, including how we handle releases. So thanks again for taking the time to express your issues. In the meantime, we are here to help with any pain you are experiencing right now. |
@thebigredgeek I'm happy to start working on it soon and get this out quickly. In the mean time, to alleviate your trouble, you can look into using https://www.npmjs.com/package/sendgrid-mailer. It's a wrapper I wrote to simplify the sending of emails with Sendgrid. My proposed API up here is pretty much an adaptation of the API in that package, so moving from it to the new version of the official package shouldn't be very time consuming. Please let me know if you have any additional feedback on the API or additions you'd like to see to make sending emails for your use cases easier. |
Thanks @adamreisnz . I hope you aren't doing this for free haha. |
Looks awesome! |
@fisch0920 I might look into that after the new API is there, but the priority at this stage is to release a new and easy to use API |
@SendGridDX do you want to await more feedback or shall we get going with this? |
I think we should get going. Based on the feedback here and other feedback on other issues on your Thanks! |
@SendGridDX Ok I'll get started. One thing though, I noticed you also added TS support, but I don't want to make my hands dirty with that, as I find that pleasing the TS community is often hard, usually filled with issues, and never very rewarding ;) Can I leave that for you or someone else on the team to implement once done? I say we focus on the pure JS implementation first, as the majority of Node users will still be using and favour ES over TS. Also, what are the requirements in terms of backward support for various Node versions? |
Thoughts on helping us support TS as we update the Mail Helper? We need to support non End-of-Life versions: https://github.com/nodejs/LTS |
@SendGridDX Hmm, Node 4 LTS has just entered maintenance phase, so I reckon we can focus on Node 6+ for this new development. Anyone on Node 4 can keep using the existing library and they are probably unlikely to switch to a new major version with a new API anyway. Thoughts? |
Hey guys, more than happy to help out - just ping me when you're nearing completion and I'll slot it into my schedule somewhere. If you have a time-box, let me know and I'll try to prioritise to meet that. |
Hi @adamreisnz, I not too familiar with what it means to enter maintenance phase. Is there a dramatic drop off in usage when this happens? And yes, that's true, we can point people to this version of the SDK if they are stuck on Node 4. |
Well it means there's still one year of maintenance support of that Node version, but this mostly pertains critical security fixes.
So personally I don't think at this stage it's necessary to keep offering support for Node 4 with the new API. |
Does anyone else following this thread have an opinion on supporting (or not) Node.js 4? |
I think v4+ is ideal. Most people don't use old versions of node in production |
@thinkingserious I have now also deprecated https://www.npmjs.com/package/sendgrid-mailer so if that's still on the list of 3rd party packages feel free to remove it as it won't be needed anymore. |
Thanks for the heads up, I've put in a PR to do so: sendgrid/docs#2788 |
Quick update, I've improved a couple of things regarding recipient handling and error response handling, as well as updated the readme with some additional examples and use cases. There's a few things still not 100% around the handling of cc/bcc's. I'm working on fixing that now and adding some tests for the helper classes to make sure they convert to the proper JSON. |
Just implemented 6.0.0-beta.3 and it works beautifully, for at least basic transactional emails:). |
Awesome, please update to beta 4 for some fixes around cc/bcc handling! |
Been following this thread for a while now. Thank you @adamreisnz, @thinkingserious, and others for their work on this. Very excited to use the new progress - but I'm having a strange issue installing the package. Just to warn - there's a very good chance I'm just being incredibly dull. When I go to
And I get a little bit more of an error when manually adding it to my
I've also tried this: Perhaps it's just a small tweak I need when installing? Thank you. |
@johndatserakis indeed it is. The first version published for this package had the beta suffix, and npm doesn't seem to handle that well when running just
or
If/when prompted, also choose the beta 4 version for the helpers package. It should work as expected again once the "full" version is released. |
Awesome thanks @adamreisnz - when using yarn the option to choose the helper version came up fine - although it didn't offer any such option in npm. It's nice that yarn does that, hopefully npm will adopt that (it might already and I'm just doing it wrong). Although as you mentioned in the future it won't be needed. Thanks again. |
No worries, I think we can release a first version soon.
…On Sun, Jul 23, 2017, 16:16 John Datserakis ***@***.***> wrote:
Awesome thanks @adamreisnz <https://github.com/adamreisnz> - when using
yarn the option to choose the helper version came up fine - although it
didn't offer any such option in npm. It's nice that yarn does that,
hopefully npm will adopt that (it might already and I'm just doing it
wrong). Although as you mentioned in the future it won't be needed. Thanks
again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#378 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAd8Qis53UVWJE5oEA2HhuVM-7ecvCAwks5sQskhgaJpZM4MyPiJ>
.
|
How's everyone that has tested the new API doing with it? Encountered any issues? Have any additional suggestions? |
Working well in my usage.
I'm loading an html email template using fs-extra, making substitutions, and adding a category. All good - although I noticed I couldn't use a $ sign in my potential substitution target in my html. No biggie though - just a simple pivot. Great work. I also really like how simple constructing and sending an email was. |
Thanks for taking the time to provide feedback @johndatserakis! |
Thanks for the feedback! @thinkingserious is the $ substitution issue related to Sendgrid's templating engine you think? |
I think so, I've seen similar issues in the past. Fortunately, it looks like that system is getting an overhaul shortly. Please bring up that issue when you chat with Matt :) With Best Regards, Elmer |
Hi @adamreisnz, |
@danteata I think the dependency for one of the packages was invalid and manually set to @thinkingserious I have pushed a change and updated the version, could you release a new beta and check if the versions in all the packages are correct? Then the install should go without hiccups. |
Thanks @adamreisnz! I should be able to dig in within the next few days. My apologies for the delay. |
Hey @adamreisnz, amazing work with this so far. Followed your initial instructions for
But it results in the following error:
@sendgrid/mail 6.0.0-beta.4 |
Hi @haydenbleasel, thank you! Yes, I fixed that bug but still waiting on @thinkingserious to release a new version. You can use beta.3 for now which doesn't have this bug. |
My apologies for the delay @adamreisnz & @haydenbleasel. I will be posting here when I get the chance to update to the new version. |
6.0.0-beta.5 has now been released, which has fixed the issue with npm/yarn complaining about invalid dependency. Thanks @thinkingserious ! |
why doesn't the default you should never have your users needing to follow a random github thread in order to figure out why your default package doesn't install properly. also, when will the default docs be updated to point to this lib instead of the more verbose version? as it stands, this library which i'm guessing 95% of your node.js users will want is way too hidden. |
@fisch0920 version 6 has not been released yet officially, as far as I know, so the beta version is the last currently available one, so you use it at your own risk. The annoying thing about npm is that it doesn't consider beta versions as stable versions, so if that's your initial version of a package, they won't install or cause some installation issues. The default package should be the old version still, called Once this version gets released as 6.0.0 it will become the default. The docs for this version have already been improved and should be up to date. @thinkingserious do you have a timeline for the release of 6.0.0? |
@adamreisnz, @fisch0920, I don't think it will take longer than early this week to officially launch v6.0.0. If you subscribe to this repo's release notes, you will get emailed when the release is official. Thanks! With Best Regards, Elmer |
Hello Everyone! v6.0.0 has been officially released! Enjoy! With Best Regards, Elmer |
Thanks, everyone! This is a huge win for the Node Sendgrid community 👍 Awesome work, @adamreisnz @thinkingserious and everyone else who contributed. Now onto sending emails... |
Awesome thanks everyone for their feedback and contributions! 🎉 |
Mail service for the Sendgrid API
This is a dedicated service for interaction with the mail endpoint of the Sendgrid API.
Basic usage
Initialization with API key
Load the library and set the API key if you haven’t set it before:
Single email to one recipient
Load the library, prepare your email data and use the
send
method:Single email to multiple recipients
The
to
field can contain an array of recipients, which will send a single email with all of the recipients in theto
field. The recipients will be able to see each other:Multiple emails to multiple recipients
If you want to send multiple individual emails to multiple recipient where they don't see each others email addresses, use
sendMultiple
instead:Note that
sendMultiple(data)
is a convenience shortcut forsend(data, true)
.Multiple single emails
The
send
method also accepts an array of email data if you want to send multiple different single emails with for example different content and sender values. This will send multiple requests (in parallel), so be aware of any API rate restrictions:Flexible email address fields
The email address fields (
to
,from
,cc
,bcc
,replyTo
) are flexible and can be any of the following:Handling success/failure
The
send
andsendMultiple
methods return aPromise
, so you can handle success and capture errors:Alternatively, pass a callback function as the last parameter:
Advanced usage
All other advanced settings are supported and can be passed in through the data object according to the expected format as per the API v3 documentation. Note that you can use either
camelCase
orsnake_case
for property names.Using transactional templates
Configure the substitution tag wrappers:
Then provide a template ID and substitutions:
Customization per recipient
To send multiple individual emails to multiple recipients with a different subject and/or substitutions, expand the
to
array as follows:Sending attachments
Attachments can be sent by providing an array of
attachments
as per the API specifications:Manually providing personalizations
Instead of using the
to
shorthand proper, you can still manually providepersonalizations
as per the API definition:Manually providing content
Instead of using the
text
andhtml
shorthand properties, you can manually use thecontent
property:Specifying time to send at
Use the
sendAt
property to specify when to send the emails (in seconds, not milliseconds):Specifying custom headers
Use the
headers
property to specify any custom headers:Specifying categories
Use the
categories
property to provide an array of categories for your email:Other options
All other options from the API definition are supported: