Simple SendGrid email adapter

[carmenlau]

Using https://github.com/sendgrid/sendgrid-nodejs

[codecov-io]

Current coverage is 90.64%

Merging #971 into master will decrease coverage by -0.78% as of 2aa08be

@@            master    #971   diff @@
======================================
  Files           73      73       
  Stmts         4384    4384       
  Branches       876     876       
  Methods          0       0       
======================================
- Hit           4008    3974    -34
  Partial         10      10       
- Missed         366     400    +34

Review entire Coverage Diff as of 2aa08be

Powered by Codecov. Updated on successful CI builds.

[gfosco]

We'd prefer that new adapters (and even the ones we have now) be moved to separate NPM packages. parse-server-sendgrid-adapter could be a good name for this one. It would be easier to require('parse-server-sendgrid-adapter') than require('parse-server/lib/Adapters/Email/SimpleSendgridAdapter') and it would put the testing/maintenance burden on the adapter developer (rather than on the core parse-server community.)

See: https://docs.npmjs.com/getting-started/publishing-npm-packages

[facebook-github-bot]

@carmenlau updated the pull request.

[noder199]

@gfosco How are we supposed to find/setup these adapters?

[gfosco]

Through NPM, and through our wiki.. I've also thought about building a search/discovery spot for parse-server modules.

[facebook-github-bot]

@carmenlau updated the pull request.

[carmenlau]

I have put this to npm and I am going to close this PR.
https://www.npmjs.com/package/parse-server-sendgrid-adapter

Search/discovery spot for parse-server modules are good idea. Thanks all for your help.

[zeluspudding]

Does the adapter support SendGrid's email templates?

[gowridev]

when sendgrid-adapter module is added to parse server git master?

[zeluspudding]

@noder199, were you able to get this adapter setup? I did however I'm clearly not initializing two variables (one the app name the other the app's domain name) as shown below in a test password reset email . I've tracked the trigger for this email down to the defaultResetPasswordEmail function located in parse server's .\node_modules\parse-server\lib\Controllers\UserController.js script within the UserController class. My question is, and perhaps @carmenlau can answer this, how does one set the appName and domain to properly generate emails? Any help is appreciated

UPDATE:
Although I've configure missing fields described above (as layed out at #554 ) I still can't actually reset passwords. Any thoughts?

[kalitas]

@zeluspudding did you manage to succeed with the password reset ? should i use the sendgrid adapter?

[zeluspudding]

I ended up waiting for the code to mature but then our project got cancelled so, unfortunately, no :( Wish I could help ya