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.

Sign up for GitHub

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

Jump to bottom

Using shipment manifest instead of line_items to handle item exchange #10

Open

octave wants to merge 4 commits into boomerdigital:master from octave:master

octave commented Sep 22, 2018

I modified the export.xml to handle exchange shipment. Prior to that when you would exchange an item in an order after customer return, it would create a new shipment but in the export.xml it would put the original item instead of the new one. I based my code on the solidus_backend view: https://github.com/solidusio/solidus/blob/master/backend/app/views/spree/admin/orders/_shipment_manifest.html.erb

octave added 3 commits

September 22, 2018 10:58


          Use shipment manifest instead of line_items to handle replaced items

59a4cc0


          Improve syntax

55639cf


          Use order.recalculate instead of deprecated order.update!

d63aa12

wycleffsean added a commit to wycleffsean/solidus_shipstation that referenced this pull request


          Update export Items

de9dd88

- use factory which actually produces line items
- Use [shipment manifest][1]
- Update [shipstation xml schema][2]

[1]: boomerdigital#10
[2]: https://help.shipstation.com/hc/en-us/articles/205928478-ShipStation-Custom-Store-Development-Guide


          Don't fetch Easypost shipments

3d10224

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet