-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[CatalogPromotion] Apply catalog promotions for product created in catalog promotions scope #13538
[CatalogPromotion] Apply catalog promotions for product created in catalog promotions scope #13538
Conversation
5179dd3
to
5594413
Compare
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingProductVariantsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Listener/ProductVariantCreatedListener.php
Outdated
Show resolved
Hide resolved
6ee7620
to
0f27fa5
Compare
0f27fa5
to
e49e3fc
Compare
@@ -250,6 +264,7 @@ protected function getDefinedElements(): array | |||
'language_tab' => '[data-locale="%locale%"] .title', | |||
'locale_tab' => '#attributesContainer .menu [data-tab="%localeCode%"]', | |||
'main_taxon' => '#sylius_product_mainTaxon', | |||
'product_taxons' => '#sylius_product_productTaxons', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use test attribute here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC we are using test attribute only in shop
And I am logged in as an administrator | ||
|
||
@ui @javascript @no-api | ||
Scenario: Applying catalog promotion on newly created simple product |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering if we should cover product creation without any variant case as well
Thank you, @ernestWarwas! 🎉 |
New products should apply for catalog promotion if falls in catalog promotion scope (eg. when creating a new product in taxon „Clothes” which applies for catalog promotion)