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

Skip to content

[CatalogPromotions] Refactor CatalogPromotionsScopeType to remove dup…#13469

Merged
GSadee merged 1 commit into
Sylius:1.11from
SirDomin:refactor-catalog-promotion-scope-form
Jan 11, 2022
Merged

[CatalogPromotions] Refactor CatalogPromotionsScopeType to remove dup…#13469
GSadee merged 1 commit into
Sylius:1.11from
SirDomin:refactor-catalog-promotion-scope-form

Conversation

@SirDomin
Copy link
Copy Markdown
Contributor

@SirDomin SirDomin commented Jan 11, 2022

…licated code

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

Remove duplicated code from event listeners and move it to generic private method

@SirDomin SirDomin requested a review from a team as a code owner January 11, 2022 11:43
@GSadee GSadee added the DX Issues and PRs aimed at improving Developer eXperience. label Jan 11, 2022
@GSadee GSadee merged commit 50bd5f9 into Sylius:1.11 Jan 11, 2022
@GSadee
Copy link
Copy Markdown
Member

GSadee commented Jan 11, 2022

Thank you, @SirDomin! 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DX Issues and PRs aimed at improving Developer eXperience.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants