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

Update product prices when customer is changed (SHUUP-2719, SHUUP-2721) #535

Merged
merged 1 commit into from
Jun 13, 2016

Conversation

gurch101
Copy link
Contributor

Fix line item pricing to reflect customer/campaign discounts for admin
order editor. Use product.get_price_info to get prices.

Refs SHUUP-2721, SHUUP-2719

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/1975/
Test FAILed.

@gurch101 gurch101 force-pushed the admin-order-customer-refresh-fix branch from 9b816d3 to 7ab8560 Compare June 10, 2016 03:03
@coveralls
Copy link

coveralls commented Jun 10, 2016

Coverage Status

Coverage decreased (-0.004%) to 86.24% when pulling 7ab85600a38b4a9a0ed11550e851b445cf74e2f5 on gurch101:admin-order-customer-refresh-fix into cfb3fa3 on shuup:master.

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/1976/
Test PASSed.

@gurch101 gurch101 force-pushed the admin-order-customer-refresh-fix branch from 7ab8560 to 3e45cf1 Compare June 10, 2016 22:49
Fix line item pricing to reflect customer/campaign discounts for admin
order editor. Use `product.get_price_info` to get prices.

Refs SHUUP-2721, SHUUP-2719
@gurch101 gurch101 force-pushed the admin-order-customer-refresh-fix branch from 3e45cf1 to 0a39dfd Compare June 10, 2016 23:03
@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/1981/
Test FAILed.

@coveralls
Copy link

coveralls commented Jun 10, 2016

Coverage Status

Coverage remained the same at 86.244% when pulling 0a39dfd on gurch101:admin-order-customer-refresh-fix into cfb3fa3 on shuup:master.

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/1983/
Test FAILed.

@tulimaki
Copy link
Contributor

retest this please

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/1984/
Test PASSed.

@suutari-ai
Copy link
Contributor

Good work, @gurch101! Thanks!

@suutari-ai suutari-ai merged commit 842f75f into shuup:master Jun 13, 2016
@gurch101 gurch101 deleted the admin-order-customer-refresh-fix branch July 1, 2016 00:10
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

5 participants