Confusing typing

[assertnotnull]

The API for Node JS is confusing and not aligned with the API:

• there's MailData.text & html but there's also MailData.content which is not optional when using MailData. The two first are not in the API doc but content is https://sendgrid.com/docs/api-reference/
• Readme mentions internal classe Mail that seems like an option to use but then the typing is not aligned with anything in the documentation. Typescript says you can't call send with Mail or MailJSON.

The current version forces me to set:

content: [
      { type: 'text/plain', value: '' },
      { type: 'text/html', value: '' },
    ],

So it's valid Typescript even though we use template ids and values.

[thinkingserious]

Hello @assertnotnull,

Great catch! When using dynamic transactional templates, we don't need to set the content values. I believe the change will need to be implemented here.

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.

With best regards,

Elmer

[acomito]

This was confusing to me as well. Seems to still be in there? I'm getting an error if I don't pass in a content field