Yeah, and since multiple addresses are simply single addresses joined by commas, we should be good.  

> To: A Group:Ed Jones <c@a.test>,joe@where.test,John <jdoe@one.test>;  

--  
Caleb Thompson
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Tuesday, March 26, 2013 at 3:00 PM, Nicolás Hock Isaza wrote:

> In lib/griddler/adapters/postmark_adapter.rb:
> > @@ -25,6 +25,15 @@ def normalize_params > > attr_reader :params > > + def full_email(contact_info) > + email = contact_info[:Email] > + if contact_info[:Name].present? > + "\"#{contact_info[:Name]}\" <#{email}>"  
> Removed quotes so the %{} won't be necessary anymore. Based on http://tools.ietf.org/html/rfc5322#appendix-A.1.1 there is no need for quotes even if there's a space on the name.
> … (#)
> -- Nicolás Hock
> On Tuesday, March 26, 2013 at 2:49 PM, Caleb Thompson wrote: In lib/griddler/adapters/postmark_adapter.rb: > @@ -25,6 +25,15 @@ def normalize_params > > attr_reader :params > > + def full_email(contact_info) > + email = contact_info[:Email] > + if contact_info[:Name].present? > + "\"#{contact_info[:Name]}\" <#{email}>" Use %{} for single-line strings needing interpolation and double-quotes. https://github.com/thoughtbot/guides/tree/master/style#ruby I still don't think we want the double quotes in the string. Do you have the RFC that specifies quotes? — Reply to this email directly or view it on GitHub (https://github.com/thoughtbot/griddler/pull/45/files#r3537497).  
>  
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/thoughtbot/griddler/pull/45/files#r3537682).
>  
>  
>