Skip to content
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

[ProductAssociationType] Fix filtering product association types by name #7143

Merged

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented Dec 21, 2016

Q A
Bug fix? yes
New feature? yes
BC breaks? no
Related tickets fixes after #7134
License MIT

@@ -0,0 +1,24 @@
@managing_product_association_types
Feature: Filtering product association types
In order to filter product association types by name or code
Copy link
Member

Choose a reason for hiding this comment

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

Do we have to be such precise? If new feature would be added should be change also description?
In order to filter product association types

Copy link
Member Author

Choose a reason for hiding this comment

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

IMO new feature =~ new file but in this case I agree with you and I will change that

@GSadee GSadee force-pushed the filtering-product-association-types branch from 8fb28b8 to 5ceb608 Compare December 21, 2016 11:44
@pjedrzejewski pjedrzejewski merged commit 8b5e349 into Sylius:master Dec 21, 2016
@pjedrzejewski
Copy link
Member

👍 👍 👍

Conclusion, everything needs to be convered! 🔥

@GSadee GSadee deleted the filtering-product-association-types branch September 22, 2017 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants