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

Apply discount to full retail price, not sale price #1008

Closed
ghost opened this issue Sep 16, 2019 · 5 comments
Closed

Apply discount to full retail price, not sale price #1008

ghost opened this issue Sep 16, 2019 · 5 comments
Assignees
Labels
💡 enhancement Ideas and suggestions
Milestone

Comments

@ghost
Copy link

ghost commented Sep 16, 2019

My client is using the site-wide Promotions > Sales option to run a near-permanent 10% off products.

They also have quite a few permanent discount codes available that offer a fixed percentage of the retail price.

For example, an item is £10.00, with the site wide sale it is available to all customers for £9. For a user with a 20% discount code, they should be able to buy the item for £8. If they enter their code at checkout, it incorrectly prices the item at £7.20 (20% off the £9).

Ideally there needs to be a way to configure a Sale or Discount to cater for this.

@lukeholder lukeholder added the 💡 enhancement Ideas and suggestions label Sep 17, 2019
@sjcallender
Copy link
Contributor

@ciceronepress For the 20% off sale, are you checking the "Ignore previous matching sales if this sale matches" setting?

Screen Shot on 2019-09-22 at 09 38 49

@lukeholder lukeholder changed the title FR: Apply discount to full retail price, not sale price Apply discount to full retail price, not sale price Sep 24, 2019
@jrrdnx
Copy link

jrrdnx commented Oct 3, 2019

@ciceronepress What version of Craft Commerce do you have installed? On 2.1.13, I have an option to apply the discount to the original item price instead of the discounted price. Is this what you're looking for?

Screen Shot 2019-10-03 at 16 43 43

@nfourtythree
Copy link
Contributor

Hi All, this is something that we are aware of and we are looking into a solution for this. Hopefully to get it into the next significant release.

@lukeholder
Copy link
Member

Should be fixed by PR #1040

@lukeholder lukeholder added this to the 2.2 milestone Oct 11, 2019
@ghost
Copy link
Author

ghost commented Oct 11, 2019

@sjcallender and @jrrdnx - I tried all combinations of these settings, but they don't provide the required option.

Many thanks @nfourtythree and @lukeholder for the fix, look forward to 2.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 enhancement Ideas and suggestions
Projects
None yet
Development

No branches or pull requests

4 participants