Nathan

Only include "completed" payments?

2

The invoice template displays _all_ payments for an order, even invalid or failed ones. As a new user to this gem, I would have expected only "completed" payments to be...

Graceful handling of missing shipping or billing address

1

My spree installation allows orders without a shipping address (i.e. digital downloaded products). Therefore, `@order.ship_address` can be `nil`, which breaks the PDF template code that prints the shipping address. You...

Doesn't play nice with CarrierWave's #move_to_cache

1

CarrierWave has an option for moving files to its cache rather than copying them: https://github.com/carrierwaveuploader/carrierwave#large-files When using `move_to_cache`, if you're using `File.open` to open the file uploaded for the original...