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

Add ability to skip premailer #66

Closed

Conversation

tjsingleton
Copy link
Contributor

Skip transforming message when the header[:premailer] is set to 'off' or 'false'.

The strings ('off' or 'false') were required, because the mail gem converted message[:premailer] = false into a premailer header w\ a nil value.

@coveralls
Copy link

Changes Unknown when pulling 5743fff on tjsingleton:allow_skipping_transform into * on fphilipe:master*.

View Details

@fphilipe
Copy link
Owner

Thanks, I’ll merge it as soon as I find some time.

@kevinthompson
Copy link

Do you think you'll find time to merge this PR in soon? It would be great to selectively disable premailer where it conflicts with other integrations such as sms-fu, or to selectively enable premailer on specific mailers.

@fphilipe
Copy link
Owner

Yes, I'll be merging this soon. Sorry for taking months. But right now I finally found some time and motivation to improve the gem. Thanks for the idea!

@fphilipe fphilipe closed this in fd6d8b2 Nov 16, 2013
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

4 participants