search

christianblais / triggerify

When X, if Y, do Z - Powerful event pipeline for your Shopify store
32 stars 8 forks source link

From serializer to attribute #122

Closed christianblais closed 2 years ago

christianblais commented 2 years ago

This is what would be required to get rid of the return if nil. Also IMO less magical, aka we're in control of the methods being called. This would allow us to get rid of the dump/load in favor of a more regular as_json approach? Anyway, food for thoughts, I don't mind either way.

/cc @tjoyal

christianblais commented 2 years ago

Closing as we don't seem to have an intent to move in that direction.