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

Add a Master SKU field to the products form #2875

Merged
merged 2 commits into from
Nov 13, 2018

Conversation

jacobherrington
Copy link
Contributor

@jacobherrington jacobherrington commented Sep 27, 2018

I have come across a use case in which the product's sku attribute needs to be
editable by the admin even in the event that the product has variants. This
commit adds a user interface element for a Master SKU which updates the
Spree::Product's sku attribute. It is visible regardless of the number of
variants, which satisfies my use case.

Solves #2874

A product with variants and a product SKU:
image

A product with variants and no product SKU:
image

A product with no variants and a product SKU:
image

jacobherrington added 2 commits September 27, 2018 06:51
I have come across a use case in which the product's sku attribute needs to be
editable by the admin even in the event that the product has variants. This
commit adds a user interface element for a Master SKU which updates the
Spree::Product's sku attribute. It is visible regardless of the number of
variants, which satisfies my use case.
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks.

Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

3 participants