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

[ProductVariant] Added versioning for all product variant details #2282

Merged
merged 2 commits into from
Mar 26, 2015

Conversation

axelvnk
Copy link

@axelvnk axelvnk commented Dec 24, 2014

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #2025
License MIT

Please do let me know if I have to adjust anything to comply to your standards although I've made sure I comply. Love to be a contributor to this project although this is a minor fix.

<gedmo:versioned/>
</field>
<field name="sold" column="sold" type="integer">
<gedmo:versioned/>
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we should log this.

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree 👍

Copy link
Author

Choose a reason for hiding this comment

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

Fine by me.

@axelvnk
Copy link
Author

axelvnk commented Mar 12, 2015

@stloyd i just made another commit and pushed to the same branch keeping in mind your comments. Should be all there now.

pjedrzejewski pushed a commit that referenced this pull request Mar 26, 2015
[ProductVariant] Added versioning for all product variant details
@pjedrzejewski pjedrzejewski merged commit 19d3c52 into Sylius:master Mar 26, 2015
@pjedrzejewski
Copy link
Member

Thank you Axel! 👍

@axelvnk
Copy link
Author

axelvnk commented Mar 27, 2015

@pjedrzejewski No problem. Could you take a look at #2627 as well?

@axelvnk axelvnk deleted the issue_2025 branch March 20, 2016 22: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

4 participants