Yes it is. It's also the Rails Way.


It's ok. I'll fork the project, rename it, credit you and the other precious devs, and carry on in a different direction.


As I wrote in the PR, I suspected that my approach deviates from where you were going. But my intended audience, I believe, is also different.

On Tue, Mar 19, 2013 at 1:22 PM, Victor Savkin <notifications@github.com>
wrote:

>>    def add_item attrs
>>      repository.create_item self, attrs
>>    end
>>  end
>>  
>>  class Item
>> -  include Edr::Model
>> -
>> -  fields :id, :name, :amount, :order_id
> I think a model should declare its fields. Pulling them automagically from the corresponding data object is very implicit.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nulogy/edr/pull/3/files#r3436098