Go Mail Helper Refactor

[thinkingserious]

Issue Summary

An interface to easily interact with the mail/send endpoint of the SendGrid v3 Web API.

The specification for this project is here.

Acceptance Criteria

All of the use cases defined here are implemented and all corresponding tests are passing.

[suchitparikh]

I got it

[thinkingserious]

@dtjm @extemporalgenome @Dharmik-Patel @deckarep @0sc @berto

If you are tagged on this message, it means we are particularly interested in your feedback :)

If you don't have the time, no worries and my apologies for the disturbance.

If you do have the time, please take a look at the proposed helper upgrade above and let us know what you think. Any and all feedback is greatly appreciated.

Thanks in advance!

[dtjm]

@thinkingserious which link did you want us to review?

[thinkingserious]

Hi @dtjm!

The relevant links are in the Issue Summary above.

Specifically, this link is the proposed specification and this link is the discussion around that proposal. And the goal is to implement these use cases.

Thanks!

With Best Regards,

Elmer

[hothero]

Hi, I have two suggestions:

1. we need to define interface and generate mock for user easily to test once they are integrating this package, can reference this issue: Make testing easier with interfaces #36

2. we need a kind of NewBatchEmail function to support send multiple recipients in the same time, more can reference here: should we have NewBatchEmail function to support multiple recipients?  #247

thanks for your reading, I can help contributing if those suggestions are accepted and you need

[hothero]

@thinkingserious I found above message is changed to work in progress, where we can follow the related progress and make contribution?

[thinkingserious]

@suchitparikh ^^

[titouanfreville]

Hello here. Is this issue still in progress ?

Having this interface would really be nice. Also, providing a default mock using testify could be a simple way to allow lot of people to use it.

Having to make a wrapper for unit testing is quite annoying 👍 :)

Thanks for your work 👍

[childish-sambino]

@titouanfreville Nope, looks like there is no one currently working on this issue.