-
Notifications
You must be signed in to change notification settings - Fork 240
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pick List per Shipment #83
Comments
@rposborne +1 |
@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? |
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. |
#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. |
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?
The text was updated successfully, but these errors were encountered: