@mtomov pointed out that this:
https://github.com/solidusio/solidus/blob/v1.3.1/core/app/models/concerns/spree/adjustment_source.rb#L7
leaves orders in an incorrect state because it doesn't run order.update! on the affected orders. Should we add that? Should we look for an alternative, like using foreign keys to prevent adjustment sources from be deleted if they are still referenced?
@mtomov pointed out that this:
https://github.com/solidusio/solidus/blob/v1.3.1/core/app/models/concerns/spree/adjustment_source.rb#L7
leaves orders in an incorrect state because it doesn't run
order.update!on the affected orders. Should we add that? Should we look for an alternative, like using foreign keys to prevent adjustment sources from be deleted if they are still referenced?