FAQ
Pull Request is out there.
https://github.com/spree/spree/pull/3389

On Sunday, July 14, 2013 7:31:09 PM UTC-6, Ryan Bigg wrote:

I have absolutely no idea... so I did some archaeology.

Turns out it's from this commit:
https://github.com/spree/spree/commit/8d34e706061e36685d325b65256dfa1256579d29done by Brian back in 2011. I don't know the reasons behind it because it's
lumped in with a ton of other changes and a vague commit message (as was
the style at the time), so I think we could get rid of it.

Could you please submit a patch to do just that Weston?


On Wed, Jul 10, 2013 at 6:21 PM, Weston Platter <weston...@gmail.com<javascript:>
wrote:
What's the main reason behind setting *ActiveRecord::Base.include_root_in_json
= true*?


https://github.com/spree/spree/blob/2-0-stable/core/lib/generators/spree/install/install_generator.rb#L103


--

Ryan Bigg
Community Manager
Spree Commerce, Inc.

Search Discussions

Discussion Posts

Previous

Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 2 of 3 | next ›
Discussion Overview
groupspree-user @
categoriesrubyonrails
postedJul 15, '13 at 1:31a
activeJul 15, '13 at 11:21p
posts3
users2
websitespreecommerce.com
irc#RubyOnRails

2 users in discussion

Ryan Bigg: 2 posts Weston Platter: 1 post

People

Translate

site design / logo © 2022 Grokbase