Proposed solution to issue #404

[tiagogodinho]

This is a proposed solution to issue #404.

The problem:

task = Task.find(1)
# => #<Task(tasks/1) title="Test" created_at="2016-08-04T20:03:42.612Z" updated_at="2016-08-06T17:00:35.649Z" id=1>
task.changes
# => {}

task = Task.all[0]
# => #<Task(tasks/1) title="Test" created_at="2016-08-04T20:03:42.612Z" updated_at="2016-08-06T17:00:35.649Z" id=1>
task.changes
# => {"title"=>[nil, "Test"], "created_at"=>[nil, "2016-08-04T20:03:42.612Z"], "updated_at"=>[nil, "2016-08-06T17:00:35.649Z"]} 

Now:

task = Task.find(1)
task.changes
# => {}

task = Task.all[0]
task.changes
# => {}
task.title = 'New title'
task.changes
# => {"title"=>["Test", "New title"]}

[lucasrenan]

👍

[edtjones]

hi @tiagogodinho @lucasrenan I've just got commit access on this repo and plan to get back on track with maintenance over time - will be in touch soon!

[zacharywelch]

@tiagogodinho this looks 💯

Before merging do you mind taking care of the specs and rebasing?

[zacharywelch]

Can we add multiple users here for consistency?

[zacharywelch]

how about?

context "for existing collections" do
  let(:users) { Foo::User.all }
  it "has no changes" do
    expect(users).to_not include(be_changed)
  end
end

[zacharywelch]

this should come in handy elsewhere 👍

[zacharywelch]

Fixed by #298. Thanks for your help on this @tiagogodinho!