spree_print_invoice icon indicating copy to clipboard operation
spree_print_invoice copied to clipboard

Pick List per Shipment

Open rposborne opened this issue 9 years ago • 4 comments

Currently it appears this gem targets an entire order. What are thoughts on changing / extending support for a print pick list for each shipment in an order?

rposborne avatar May 03 '15 16:05 rposborne

@rposborne +1

futhr avatar May 03 '15 17:05 futhr

@futhr How should we go about this. It feels weird to extend the API shipments controller to render out out a get PDF. Or should we outright add a shipment controller in backends scope?

rposborne avatar May 04 '15 13:05 rposborne

I would definitely vote for a separate shipments controller. Therefore we can keep the order based documents and can have additional for shipments. We need then to also patch the shipment model, since as with the order, that's where the PDF rendering happens. So extracting the PDF generation out into a module would be best option IMO.

tvdeyen avatar May 04 '15 13:05 tvdeyen

https://github.com/spree-contrib/spree_print_invoice/pull/87 will allow you to create PDFs for arbitrary Spree Models, among them Shipments. Because yes, what you want makes complete sense: A packaging slip for a shipment.

mamhoff avatar Jul 07 '15 14:07 mamhoff