solidusio / solidus

🛒 Solidus, the open-source eCommerce framework for industry trailblazers.
https://solidus.io
Other
5.01k stars 1.29k forks source link

Add classes instead of strings to `ActiveRecord.yaml_column_permitted_classes` for Psych 5 #5894

Closed tvdeyen closed 2 hours ago

tvdeyen commented 2 hours ago

In https://github.com/solidusio/solidus/pull/5788 we allowed to use Psych 5. In Psych 5 the permitted classes need to be actual classes not Strings.

See https://stackoverflow.com/questions/73958140/correct-way-to-set-yaml-column-permitted-classes-for-a-custom-class

tvdeyen commented 2 hours ago

We already use classes. My bad