I'm fine with that too :)

Josh


On Mon, Jan 6, 2014 at 11:56 AM, Matt Drees <notifications@github.com>wrote:

> In app/controllers/contacts_controller.rb:
>
> > +                                     addresses_attributes: [:remote_id, :master_address_id, :location, :street, :city, :state, :postal_code, :country, :primary_mailing_address, :_destroy, :id],
> > +                                     people_attributes: [:first_name, :last_name, :legal_first_name, :title, :suffix, :deceased, :birthday_month, :birthday_day,
> > +                                                         :birthday_year, :marital_status, :anniversary_month, :anniversary_day, :anniversary_year, :_destroy, :id,
> > +                                                         {
> > +                                                           email_addresses_attributes: [:email, :primary, :_destroy, :id],
> > +                                                           phone_numbers_attributes: [:number, :location, :primary, :_destroy, :id],
> > +                                                           linkedin_accounts_attributes: [:url, :_destroy, :id],
> > +                                                           facebook_accounts_attributes: [:url, :_destroy, :id],
> > +                                                           twitter_accounts_attributes: [:screen_name, :_destroy, :id],
> > +                                                           pictures_attributes: [:image_cache, :primary, :_destroy, :id],
> > +                                                           family_relationships_attributes: [:related_person_id, :relationship, :_destroy, :id]
> > +                                                         }
> > +                                     ]
> > +                                    }
> > +   )
> > +  end
>
> is this a ruby idiom? It just seems hard for me to read with so much stuff
> being so indented. I think it'd be easier to read if it was formatted the
> way json typically is.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/CruGlobal/mpdx/pull/45/files#r8669934>
> .
>