Adding appropriate tests for the new backend cart
Removing unecessary changes from the API variants controller
Not allowing you to edit the cart or shipments after shipping
Removing unnecessary code for getting the total_on_hand for a variant through the API
Extracting the advanceOrder function into a function in the Spree namespace so that both the shipmentsJS and line_itemsJS can share it
Advancing the order after we add variants on the cart page. If we have already created shipments, then when we edit a line item the order gets reset to the address state. This call pushes it back to the payment state and destroys the old shipments, recreating them to match the new line item state.
Renaming the edit_line_items method to cart. Also since they were on the same line, assigning a proper icon for the cart page
In checkout.rb we first create_proposed_shipments but here we check if addresses are valid. If not we need to destroy_all instead of delete_all so that the inventory_units that were created by create_propsed_shipments will also be destroyed and not clutter up the DB
Eliminate unnecessary instance var
Removing the ability to change variant quantities on the Admin order shipments page
Fixing the Split tool. IF success we reload the page, else display a generic error. Would like to in the future provide more specific errors. Not allowing splits to the same shipment nor to a shipment or location that does not have the stock and is NOT backorderable. Next commit will have a boatload of automated tests.
Fixing a callout in our transfer/split code for the admin order editor