mysociety / petitions

Petitions website code, formerly used by Number 10, still used by various local councils
Other
25 stars 5 forks source link

Simplify petemaild #99

Closed dracos closed 4 years ago

dracos commented 4 years ago

Fixes #42. Does two things - removes the domain resolver, assuming there's a localhost email server. And then removes the email sender, replacing it with Email::Sender. Could do either/or of those if there were some reason we didn't want to do both. The test suite passed on firefly.

dracos commented 4 years ago

"It looks fine - although this is a pretty large program, and I'm not hugely familiar with all its parts." - yeah, that's one good reason to do this, I think, as I find it pretty hard to follow too!