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.

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

Show bulk discounts only if enabled through store settings. #1310

Merged
merged 1 commit into from
Jul 13, 2018
Merged

Show bulk discounts only if enabled through store settings. #1310

merged 1 commit into from
Jul 13, 2018

Conversation

Ubersmake
Copy link
Contributor

@Ubersmake Ubersmake commented Jul 12, 2018

What?

Cornerstone shows bulk discounts available on all products that have rules associated with them, regardless of the "Enable Bulk Discount?" setting in Display Settings in Store Settings > Display.

This is reliant on a change to expose settings.bulk_discount_enabled to the Stencil context: https://github.com/bigcommerce/bigcommerce/pull/25352

If Bulk Discounts are enabled, bulk discounts will show on product pages with bulk discount rules associated with them, as they do now.

If Bulk Discounts are disabled, bulk discounts will not show up, even if a product has a bulk discount rule applied to it.

Tickets / Documentation

Screenshots

Before

Product Without bulk discounts:

beforewithout

Product With bulk discounts, Bulk discounts Disabled:

beforewithdisabled

Product With bulk discounts, Bulk discounts Enabled:

beforewithenabled

After

Product Without bulk discounts:

afterwithout

Product With bulk discounts, Bulk discounts Disabled:

afterwithdisabled

Product With bulk discounts, Bulk discounts Enabled:

afterwithenabled

Warnings

If a theme with this change is somehow applied to a store without the Bigcommerce change that exposes settings.bulk_discount_enabled, bulk pricing will not appear under any circumstances because the Handlebars if is wrapped around that div.

@bigbot
Copy link

bigbot commented Jul 12, 2018

Autotagging @bigcommerce/storefront-team @davidchin

@Ubersmake Ubersmake merged commit f3910d4 into bigcommerce:master Jul 13, 2018
@Ubersmake Ubersmake deleted the STRF-4969 branch July 13, 2018 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants