-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Admin] Changed product code to variant code on orders show page #6128
[Admin] Changed product code to variant code on orders show page #6128
Conversation
NoResponseMate
commented
Sep 15, 2016
Q | A |
---|---|
Bug fix? | yes |
New feature? | yes |
BC breaks? | no |
Related tickets | fixes #6116 |
License | MIT |
@@ -25,7 +25,8 @@ Feature: Seeing order item detailed data | |||
Scenario: Seeing details of item in one row | |||
Given I view the summary of the order "#00000666" | |||
When I check "Iron Man T-Shirt" data | |||
Then its unit price should be $39.00 | |||
Then its code should be "IRON_MAN_T-SHIRT" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is it defined?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To verify this fix is correct we should add two variants of the same product and validate their codes.
58dad00
to
2d5091f
Compare
2d5091f
to
c9c9510
Compare
And it belongs to "T-Shirts" tax category | ||
And the store has a product "Marvel T-Shirt" | ||
And this product has "Iron Man T-Shirt" variant priced at "$49.00" | ||
And this product has "Thor T-Shirt" variant priced at "$39.00" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...the "XXX" variant...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not in scope of this PR.
Besides most of our steps need a hardcore semantical rework
Thanks Janek! 👍 |
…duct-variant-code [Admin] Changed product code to variant code on orders show page