Skip to content
This repository was archived by the owner on Aug 30, 2018. It is now read-only.

[Timber] add cart discounts#522

Merged
matcaissy merged 1 commit intomasterfrom
timber-add-cart-discounts
Mar 21, 2016
Merged

[Timber] add cart discounts#522
matcaissy merged 1 commit intomasterfrom
timber-add-cart-discounts

Conversation

@matcaissy
Copy link

@matcaissy matcaissy self-assigned this Mar 17, 2016
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add the price right in this string? Some other languages might prefer the price and then savings text?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes a lot of sense to me; I would say this warrants a change.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@matcaissy matcaissy force-pushed the timber-add-cart-discounts branch from 6152ae1 to 15fdc67 Compare March 21, 2016 18:11
@matcaissy
Copy link
Author

@NathanPJF squashed and updated with price in the locale strings. Mind another 👀 Thx :)

@carolineschnapp
Copy link
Contributor

🎩

matcaissy pushed a commit that referenced this pull request Mar 21, 2016
@matcaissy matcaissy merged commit d123269 into master Mar 21, 2016
@matcaissy matcaissy deleted the timber-add-cart-discounts branch March 21, 2016 21:14
@suture
Copy link

suture commented Mar 22, 2016

When does the discount show up on the Cart page?

I've tested adding a discount promo code on the checkout page, then navigating back to the cart page, but it nothing relating to the discount shows up.

No discount title or discounted price.

@shoshanna
Copy link

I have the same question. I'm not seeing the discount on the cart yet, how is it supposed to work?

@suture
Copy link

suture commented Apr 6, 2016

Just a guess. Since the #2953 link above goes to a 404 error page, I assume that it's a fix for a new feature on a private Github page.

@carolineschnapp
Copy link
Contributor

This pull request isn't related to discount codes as you know them. It is to support automated cart savings which are only available to Plus merchants who use Shopify Scripts: https://www.shopify.com/enterprise/98548614-introducing-shopify-scripts-checkout-api-a-giant-leap-forward-for-hosted-ecommerce.

@Shopify Shopify locked and limited conversation to collaborators Apr 6, 2016
vendor: cartItem.vendor
vendor: cartItem.vendor,
linePrice: Shopify.formatMoney(cartItem.line_price, settings.moneyFormat),
originalPrice: Shopify.formatMoney((cartItem.line_price + cartItem.total_discount), settings.moneyFormat),
Copy link
Contributor

@NathanPJF NathanPJF Jun 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is talking about cartItem.line_price, this can be:

originalLinePrice: Shopify.formatMoney(cartItem.original_line_price, settings.moneyFormat)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments