fixes broken mail configuration #752

Merged
merged 2 commits into from Dec 23, 2016

Projects

None yet

2 participants

@jetware
Contributor
jetware commented Oct 20, 2016

This pull request includes two patches:

  • fixes code to use actual ActiveMailer API
  • changed mail.yml.example with syntax corresponding to actual ActiveMailer API

NB: The change in using key from 'method' to 'delivery_method' should not break settings for existing installation because they are already broken and use default :sendmail settings due fallback on the 'server_settings' assignment exception.

jetware added some commits Oct 20, 2016
@jetware jetware fixes #751
Reads from mail.yaml value of delivery_method and settings for each
method supported, and assings them to ActionMailer options.
222e533
@jetware jetware issue #751, mail configuration example rewritten 5887dc0
@mvz mvz merged commit 34ed1a5 into publify:master Dec 23, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@mvz
Contributor
mvz commented Dec 23, 2016

Thanks, @jetware!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment