state-machines / state_machines-activerecord

StateMachines Active Record Integration
https://github.com/state-machines/state_machines-activerecord
MIT License
401 stars 83 forks source link

#101 docs(models): examples show models inheriting ApplicationRecord #100

Closed casaper closed 2 years ago

casaper commented 2 years ago

Since Rails 5.0 or 5.1 rails explicitly asks us to inherit from < ApplicationRecord and not < ActiveRecord::Base.

I Know it is kind of a nit pick. But this code style flaw in the examples still feels kind of weird.

And the fix is not all that large, right? :wink:

PS:

And then I also spotted validates_presence_of which is deprecated as well since ... never mind.

Also, a simple fix.

Filling my own issue #101