laserlemon / vestal_versions

Keep a DRY history of your ActiveRecord models' changes
MIT License
933 stars 229 forks source link

Fixes for newer ActiveRecord #82

Closed futurechimp closed 11 years ago

futurechimp commented 12 years ago

Hi, there are a few problems with Vestal Versions when used with the newest ActiveRecord - here are some fixes.

I've also (more controversially) symbolized keys coming out of the Changes#changes_between method, that might screw things up - I can revert this if you like.

I'm just about to release a simple gem for doing diffs on Vestal Versions (using another gem called Diffy). For the moment I guess I'll just depend on my fork, but it'd be nice to get back to the mainline if possible.

dreamr commented 11 years ago

Still missing some elements, but it has been handled in the latest master. Thanks!