@aleksa-krolls , I've tested this locally but want your eyes on it as well. I think this is a more intuitive way to handle empty strings (testing them directly). Beyond that, each time we were using oscarValue(c.external_id) ? null : 'some value' i've opted for isUpdate ? null : 'some value' as it feels simpler for humans.
@aleksa-krolls , I've tested this locally but want your eyes on it as well. I think this is a more intuitive way to handle empty strings (testing them directly). Beyond that, each time we were using
oscarValue(c.external_id) ? null : 'some value'
i've opted forisUpdate ? null : 'some value'
as it feels simpler for humans.