Skip to content
This repository was archived by the owner on Mar 2, 2020. It is now read-only.

enable passing optional email in query string and sending email to that ... #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JamesLavin
Copy link

...address

@jeremy
Copy link
Member

jeremy commented Mar 13, 2014

Hey @JamesLavin - This is cool! It's pushing beyond the mail_view lib's aims, though. Would need a compelling need to justify expanding its scope. How are you using this? Why'd you need to add it?

@JamesLavin
Copy link
Author

Thanks for taking time to look at and comment on my PR, @jeremy! Apologies for not noticing your reply till now.

Here at Hedgeye, we have a saying: "It's not an email until it's an email." When our designers play around with email formatting, we change text, etc., we want to actually send out test emails and view them on different devices and in different email clients to confirm that everything displays correctly. Viewing an email in a browser is great for development, but -- our email czar insists -- it's not sufficient, given the quirks in how various email clients render emails.

We've been using the modified version of mail_view since I created it, and we have found it helpful. But I appreciate that this may be out of scope. If so, no worries.

Regardless, thanks for creating mail_view!

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

Successfully merging this pull request may close these issues.

2 participants