search

westonganger / active_snapshot

Simplified snapshots and restoration for ActiveRecord models and associations with a transparent white-box implementation
MIT License
103 stars 16 forks source link

Fix ArgumentError when model doesn't define has_snapshot_children #19

Closed danderozier closed 2 years ago

danderozier commented 2 years ago

Resolves #4

westonganger commented 2 years ago

I think I would rather use respond_to? instead of defining an attr_reader. What do you think?

westonganger commented 2 years ago

Can you also add a test case for this. Would be great.

westonganger commented 2 years ago

Did you want to update this PR? or shall I make the changes myself.

westonganger commented 2 years ago

Replaced by PR #20

westonganger commented 2 years ago

v0.2.4 is now released which contains the fix for this.