Comment by snarfy

    bool productPricingUpdatedAfterDeliveryFailedButEmailSent = false;

I've seen variables with names like the above. It would have been better explained in a comment and used a short variable name. Unless you like typing it forces you to use an editor with autocomplete (which you are crazy not to use one), and IMO pollutes the code. Variables should be easy to type and read. It's a trade off.