Skip to content
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

Fix Spree::Shipment#item_cost for split shipments #18

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix Spree::Shipment#item_cost for split shipments #18

wants to merge 4 commits into from

Commits on Feb 1, 2019

  1. Rename Spree::Shipment#manifest to shipping_manifest_items 

    `manifest` method returns Spree::ShippingManifest#items, this commit is
going to rename it to `shipping_manifest_items`.

I also refactored it to use the memoize shipping_manifest method.
    @DanielePalombo
    DanielePalombo committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    0cf96f6 View commit details
    Browse the repository at this point in the history

  2. Rename Spree::Carton#manifest to shipping_manifest_items 

    `manifest` method returns Spree::ShippingManifest#items, this commit is
going to rename it to `shipping_manifest_items`.

I also refactored it to use the memoize shipping_manifest method.
    @DanielePalombo
    DanielePalombo committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    fb28cce View commit details
    Browse the repository at this point in the history

  3. Rename manifest_for_order to shipping_manifest_for_order 

    This commit is going to rename the method `manifest_for_order` to
`shipping_manifest_for_order`
I also refactored it to use the memoized `shipping_manifest` method.
    @DanielePalombo
    DanielePalombo committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    aaa2411 View commit details
    Browse the repository at this point in the history

  4. Sum Spree::ShippingManifest::ManifestItem.item_cost 

    This commit is going to add `item_cost` calculation, all these item costs will be summed.
    @DanielePalombo
    DanielePalombo committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    1d321f2 View commit details
    Browse the repository at this point in the history