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.

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

Pass request params to ActionMailer::Preview #28244

Merged

Conversation

ixti
Copy link
Contributor

@ixti ixti commented Mar 1, 2017

Allows passing query params to email previews:

http://localhost:3000/rails/mailers/user_mailer/password_recovery?email=alice@foobar.com

Second attempt of #25193

@ixti ixti force-pushed the improve/action-mailer-preview-params branch from 5633ca9 to 8e6c6d8 Compare March 1, 2017 19:54
Copy link
Member

@rafaelfranca rafaelfranca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I'll merge it after the Rails 5.1 branch is created.

@rafaelfranca rafaelfranca added this to the 5.2.0 milestone Mar 1, 2017
@ixti
Copy link
Contributor Author

ixti commented Mar 1, 2017

@rafaelfranca Thanks!

@thinkclay
Copy link

The 5.1 branch exists now. Will we see this merged in the next release?

@rafaelfranca rafaelfranca merged commit f680664 into rails:master Apr 27, 2017
@ixti ixti deleted the improve/action-mailer-preview-params branch April 27, 2017 15:39
@ixti
Copy link
Contributor Author

ixti commented Apr 27, 2017

Thanks! 🙇

@khoan
Copy link

khoan commented Nov 29, 2017

hasn't made it into Rails 5.1.4 yet.

@rafaelfranca
Copy link
Member

This is going to be released in 5.2.0

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

Successfully merging this pull request may close these issues.

None yet

5 participants