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

[Docu] Quantity Price Rules #1716

Merged
merged 5 commits into from
Sep 13, 2021
Merged

[Docu] Quantity Price Rules #1716

merged 5 commits into from
Sep 13, 2021

Conversation

lukadschaak
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets -

Documenting the usage of quantity price rules.

With help by @Theux94

Specific Prices are basically the same as Product Price Rules, but are configured directly on the
product and therefore are only applied on that product.

It is recommended to have at least one specific price, which is always applicable. For example a
Copy link
Member

Choose a reason for hiding this comment

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

it is not, why should it?


Quantity price rules are only applied to the price in context of a cart. In other cases the
price calculator ignores them. So it is recommended to not have quantity price rules alone,
but to have at least a default specific price (or a store values entry).
Copy link
Member

Choose a reason for hiding this comment

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

Don't recommend things here.

@@ -19,6 +19,8 @@ CoreShop Price Calculation consists of 3 different prices:
- **Discount**: Discount from promotions
- **Price**: Retail Price or Discount Price (if available) minus discount rules

To get a non-zero price, the calculation needs at least a matching specific price rule, or a store value to work.
Copy link
Member

Choose a reason for hiding this comment

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

please remove. the default is ALWAYS to have a store-value. price

@lukadschaak
Copy link
Contributor Author

Updated, please check changes.

@lukadschaak
Copy link
Contributor Author

push @dpfaffenbauer Still interested in this part of documentation?

@dpfaffenbauer
Copy link
Member

@lukadschaak yes, sorry, I am on holiday right now and had lots todo before I left. Will review this when I am back.

@lukadschaak
Copy link
Contributor Author

Ohh holiday is nice! Keep on refreshing!! :)

@dpfaffenbauer dpfaffenbauer added this to the 2.2.10 milestone Sep 13, 2021
@dpfaffenbauer dpfaffenbauer merged commit 48a8bab into coreshop:2.2 Sep 13, 2021
@dpfaffenbauer
Copy link
Member

thanks a lot

@lukadschaak lukadschaak deleted the patch-2 branch September 28, 2021 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants