-
-
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
Catalog promotions action #13029
Catalog promotions action #13029
Conversation
SirDomin
commented
Aug 30, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | |
Deprecations? | no |
Related tickets | |
License | MIT |
src/Sylius/Bundle/CoreBundle/Migrations/Version20210826063828.php
Outdated
Show resolved
Hide resolved
src/Sylius/Component/Promotion/Model/CatalogPromotionInterface.php
Outdated
Show resolved
Hide resolved
features/promotion/managing_catalog_promotions/creating_catalog_promotion.feature
Outdated
Show resolved
Hide resolved
features/promotion/managing_catalog_promotions/editing_catalog_promotion.feature
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Resources/config/serialization/CatalogPromotion.xml
Outdated
Show resolved
Hide resolved
...Sylius/Bundle/PromotionBundle/Resources/config/doctrine/model/CatalogPromotionAction.orm.xml
Outdated
Show resolved
Hide resolved
->addDefaultsIfNotSet() | ||
->children() | ||
->scalarNode('model')->defaultValue(CatalogPromotionAction::class)->cannotBeEmpty()->end() | ||
->scalarNode('interface')->defaultValue(CatalogPromotionActionInterface::class)->cannotBeEmpty()->end() |
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.
Isn't needed anymore
->scalarNode('interface')->defaultValue(CatalogPromotionActionInterface::class)->cannotBeEmpty()->end() |
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.
Indeed 👍 but to be consistent with other configurations I would leave it for now and remove altogether later 🖖
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Context/Api/Admin/ManagingCatalogPromotionsContext.php
Outdated
Show resolved
Hide resolved
tests/Api/Responses/Expected/admin/catalog_promotion/post_catalog_promotion_response.json
Show resolved
Hide resolved
8a348bd
to
9a1c189
Compare
763187c
to
c7af95f
Compare
@@ -18,4 +18,5 @@ | |||
|
|||
interface CatalogPromotionInterface extends BaseCatalogPromotionInterface, ChannelsAwareInterface | |||
{ | |||
public const TYPE_CONTAINS_VARIANTS = 'contains_variants'; |
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.
This type should be on CatalogPromotionRuleInterface in Core, and it has been already changed to for_variants
Thanks, @SirDomin! 🥇 |
…on (Zales0123) This PR was merged into the 1.11-dev branch. Discussion ---------- | Q | A | --------------- | ----- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Related tickets | based on #13029 | License | MIT Commits ------- 7a01498 Use CatalogPromotion actions for discounts application d0afff7 PR review fixes