Closed yuenmichelle1 closed 1 month ago
Apologies for big-ish PR, but change should be straight forward.
See similar PR on panoptes that was done related to rails upgrades here: https://github.com/zooniverse/panoptes/pull/3979 (Can be used as context when reviewing)
https://guides.rubyonrails.org/upgrading_ruby_on_rails.html#active-record-models-now-inherit-from-applicationrecord-by-default
Some background on why this change was made
https://github.com/rails/rails/pull/22567#discussion-diff-47831156 https://stackoverflow.com/questions/37359527/why-rails-5-uses-applicationrecord-instead-of-activerecordbase
Apologies for big-ish PR, but change should be straight forward.
See similar PR on panoptes that was done related to rails upgrades here: https://github.com/zooniverse/panoptes/pull/3979 (Can be used as context when reviewing)
https://guides.rubyonrails.org/upgrading_ruby_on_rails.html#active-record-models-now-inherit-from-applicationrecord-by-default
Some background on why this change was made
https://github.com/rails/rails/pull/22567#discussion-diff-47831156 https://stackoverflow.com/questions/37359527/why-rails-5-uses-applicationrecord-instead-of-activerecordbase