-
-
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
[CartPromotion][API][UI] Apply on discounted flag added #13328
[CartPromotion][API][UI] Apply on discounted flag added #13328
Conversation
AdamKasp
commented
Nov 24, 2021
•
edited
Loading
edited
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
License | MIT |
91469cd
to
18c4f06
Compare
src/Sylius/Behat/Context/Api/Admin/ManagingPromotionsContext.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Migrations/Version20211124101833.php
Outdated
Show resolved
Hide resolved
"rules": [], | ||
"actions": [], | ||
"applyOnDiscounted": false, | ||
"createdAt": @string@, |
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.
"createdAt": @string@, | |
"createdAt": @string@.isDateTime(), |
"actions": [], | ||
"applyOnDiscounted": false, | ||
"createdAt": @string@, | ||
"updatedAt": @string@ |
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.
"updatedAt": @string@ | |
"updatedAt": @string@.isDateTime() |
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.
it doesn't work in this way, the checker see it as a string @string@.isDateTime()
18c4f06
to
890948d
Compare
src/Sylius/Bundle/PromotionBundle/Resources/config/doctrine/model/Promotion.orm.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/PromotionBundle/Resources/config/doctrine/model/Promotion.orm.xml
Outdated
Show resolved
Hide resolved
9dc073e
to
105b62f
Compare
src/Sylius/Bundle/PromotionBundle/Resources/config/doctrine/model/Promotion.orm.xml
Outdated
Show resolved
Hide resolved
features/promotion/managing_promotions/adding_promotion.feature
Outdated
Show resolved
Hide resolved
5af5b6e
to
5d007dc
Compare
src/Sylius/Bundle/PromotionBundle/Resources/translations/messages.en.yml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/PromotionBundle/Resources/translations/messages.en.yml
Outdated
Show resolved
Hide resolved
629c8b1
to
290c5dd
Compare
290c5dd
to
81539f5
Compare
Thanks, Adam! 🥇 |
…nly on non discounted products (GSadee, SirDomin) 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 #13328 | License | MIT <!-- - Bug fixes must be submitted against the 1.10 branch - Features and deprecations must be submitted against the master branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> Commits ------- 4413793 [CartPromotion][CatalogPromotion][Behat] Add scenarios for receiving discount only on non discounted products a65561e [CartPromotion][CatalogPromotion][Behat] Implement scenarios for receiving discount only on non discounted products 3556f47 [CartPromotion][CatalogPromotion][Behat] Fixes after rebase with changing the name of field 1f1e798 [CartPromotion][CatalogPromotion] Receiving discount only on non discounted products for unit cart promotions 372365b [WIP][CartPromotion][CatalogPromotion] Receiving discount only on non discounted products for fixed order cart promotions e735927 [CartPromotion] handle fixed and percentage order discount 9ce4cb8 [MinimumPrice][ApplyOnDiscounted] merge solutions for apply on discount and minimum prices