Remove Spree::OrderObserver.

It's not clear why this observer was necessary, since there is already an observer on payment which should run after_finalize.

If anyone has a clue why this was needed, I'd be keen to hear.