enable passing optional email in query string and sending email to that ...#65
enable passing optional email in query string and sending email to that ...#65JamesLavin wants to merge 1 commit intobasecamp:masterfrom
Conversation
|
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? |
|
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! |
...address