3.1.0 is including breaking changes without error and is a beta endpoint?!
See original GitHub issueHi,
I’m really disappointed with the changes in the API, you did breaking changes without telling anybody.
The problem here is that I’m using “To” & “Bcc” old format SMTP style
and it’s not throwing an error when I use it with the latest v3.1.0 of node-mailjet. I only had people complaining that they didn’t receive emails but that was weird for me as the “Bcc” email was sent to me successfully.
I just figured out the problem when I opened the API Guides and found that https://dev.mailjet.com/beta/#send-api-v3-to-v3-1
To, Cc, Bcc The SMTP style description of the recipients is replaced by a collection of recipients objects (with Email and Name property), nested in array.
Also the 3.1 is marked as BETA here, this is really confusing, you shouldn’t use a BETA endpoint as people are using this library in production and without any notice!
Also the tests are too light for this library, only “Recipients” is tested and not “To” & “Bcc” (See https://dev.mailjet.com/guides/#sending-a-basic-email)
Please do better tests and throw an error if the “To” & “Bcc” are not correct instead of just sending half of the emails…
Issue Analytics
- State:
- Created 6 years ago
- Comments:6 (3 by maintainers)
Top GitHub Comments
Got it, let’s pursue there.
Thank you for the fast support 😃 Here it is: N° 681767