Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[CatalogPromotion] Secure that the catalog promotion cannot be edited#13195

Merged
AdamKasp merged 2 commits into
Sylius:masterfrom
GSadee:denying-edition-catalog-promotion-during-processing
Oct 13, 2021
Merged

[CatalogPromotion] Secure that the catalog promotion cannot be edited#13195
AdamKasp merged 2 commits into
Sylius:masterfrom
GSadee:denying-edition-catalog-promotion-during-processing

Conversation

@GSadee
Copy link
Copy Markdown
Member

@GSadee GSadee commented Oct 12, 2021

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Related tickets
License MIT

@GSadee GSadee added Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). Admin AdminBundle related issues and PRs. API APIs related issues and PRs. labels Oct 12, 2021
@GSadee GSadee requested a review from a team as a code owner October 12, 2021 11:11
Then "PHP T-Shirt" variant should be discounted

@api @ui
Scenario: Being unable to edit catalog promotion if it is currently being processed
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WDYT about one more scenario Being able to edit catalog promotion after processing is completed?

@AdamKasp AdamKasp merged commit 7945169 into Sylius:master Oct 13, 2021
@AdamKasp
Copy link
Copy Markdown
Contributor

Thank you, Grzegorz! 🎉

@GSadee GSadee deleted the denying-edition-catalog-promotion-during-processing branch October 13, 2021 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Admin AdminBundle related issues and PRs. API APIs related issues and PRs. Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants