craftcms / commerce

Fully integrated ecommerce for Craft CMS.
https://craftcms.com/commerce
Other
218 stars 170 forks source link

[5.x]: New products/variants are not promotable by default #3571

Closed mikesnoeren closed 2 months ago

mikesnoeren commented 2 months ago

What happened?

Description

As per the documentation, the promotable option should be enabled by default. In Craft 5 + Commerce 5 this is not the case.

Steps to reproduce

1.Make a product 2.See see the lightswitch is not flipped by default.

Craft CMS version

5.2.5

Craft Commerce version

5.0.11.1

PHP version

No response

Operating system and version

No response

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

-

linear[bot] commented 2 months ago

PT-1930 [5.x]: New products/variants are not promotable by default

nfourtythree commented 2 months ago

Hi @mikesnoeren

Thank you for reporting this. We have made an issue in the docs to be updated to make sure they explain how the system is working.

We have also added the promotable property to the options for showing in the variant index. This will allow you to have better visibility over which variants have been set to be profitable. This change will be included in the next release of Commerce.

We will take the feedback on board and see if there is a way (maybe a setting) to change the behaviour for whether a new variant is promotable by default.

Thanks!

mikesnoeren commented 2 months ago

Hi @nfourtythree, thanks for creating the issue in the docs.

From my perspective, enabling the promotable property by default would be the most effective approach. Any necessary (conditional) rules for promotions should, in my opinion, be managed within the promotion itself.

Introducing this setting seems to add unnecessary complexity to the already intricate promotions functionality.

Additionally, I find it quite unconventional to restrict certain products from being eligible for promotions. Having worked on multiple webshops, we have never encountered a need for this restriction.

nfourtythree commented 2 months ago

Hi @mikesnoeren

Again, thank you for this feedback. We will take it on board when making decisions regarding this functionality.

Thanks!