Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Provide a better default ActionMailer delivery_method in development env #516

Closed
gylaz opened this issue Feb 10, 2015 · 3 comments
Closed

Provide a better default ActionMailer delivery_method in development env #516

gylaz opened this issue Feb 10, 2015 · 3 comments

Comments

@gylaz
Copy link
Contributor

gylaz commented Feb 10, 2015

It seems that Rails 4.2 defaults to :deliver as the default ActionMailer delivery_method. Thus, emails will try to deliver via sendmail. Seems not very useful in most dev setups. Should we default that to :test in development environment instead?
@jferris
Copy link
Contributor

jferris commented Feb 10, 2015

I think we should use :test, yes.

@croaky
Copy link
Contributor

croaky commented Feb 14, 2015

Agreed.

gylaz pushed a commit that referenced this issue Feb 16, 2015
Set email deliver method to :test for development
0f46593 
`:test` method will not send the email, but instead print it to the logs. This
will allow faster development cycle, since there is no need to setup
smtp/sendmail when email is involved.

Closes #516
@gylaz gylaz mentioned this issue Feb 16, 2015
Closed
@tute
Copy link
Contributor

tute commented Feb 17, 2015

Closing in favor of PR.

@tute tute closed this as completed Feb 17, 2015
gylaz pushed a commit that referenced this issue Feb 17, 2015
@tute
Set email deliver method to :test for development
8b45fd4 
`:test` method will not send the email, but instead print it to the logs. This
will allow faster development cycle, since there is no need to setup
smtp/sendmail when email is involved.

Closes #516
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set email deliver method to :test for development thoughtbot/suspenders
4 participants
@croaky @jferris @tute @gylaz