amberframework / granite

ORM Model with Adapters for mysql, pg, sqlite in the Crystal Language.
MIT License
296 stars 87 forks source link

Fixing a timestamp issue to support Crystal 1.0.0+ #465

Closed crimson-knight closed 2 years ago

crimson-knight commented 2 years ago

As @Blacksmoke16 suggested here, I changed the #touch method to use #set_timestamps instead of trying to directly update the @updated_at attribute.

I couldn't get the test suite to run locally, however, I made a test project that duplicated the issue described here and on the Amber repo and changed the info for Granite to point to my forked version with the change here and amber successfully compiled and displayed the welcome page.

I think this is a success!