aws-ses-wp-mail icon indicating copy to clipboard operation
aws-ses-wp-mail copied to clipboard

Published 20 hours ago • verified publisher icon humanmade

[Refactor] Use PHPMailer

Open faishal opened this issue 6 years ago • 2 comments

  • Removed wp_mail pluggable method, hook into wp_mail & PHPmailer
  • Added SES_PHPMailer custom class by extending WordPress PHPMailer
  • Use custom Mailer method to send the email via aws ses

faishal avatar Apr 05 '18 05:04 faishal

@faishal I think this is very cool! I'm really torn on whether to go for this refactor or not. It seems a lot more elegant and simpler that what we have now. @rmccue third opinion on taking this approach? I'm particularly sensitive to not breaking anything. I wonder if we could have an integration test of some sort to make sure there's no regressions here too.

joehoyle avatar Apr 10 '18 16:04 joehoyle

Don't know how it compares, but in case it got unnoticed: @diegoh's (MIT-licenced) https://github.com/creativecity/wp-aws-ses

drzraf avatar Apr 27 '18 15:04 drzraf