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

#161268604 Customer/Admin should be able to price of orders #77

Merged
merged 2 commits into from Oct 16, 2018

Conversation

akhilome
Copy link
Owner

What does this PR do?

adds total order price to response objects coming from routes hitting the orders table

Description of Task to be completed?

  • add price column to orders table
  • automatically calculate price when a new order is placed.
  • modify tests to reflect necessary changes
  • update code to make tests pass

How should this be manually tested?

N/A

What are the relevant pivotal tracker stories?

#161268604

Screenshots

image

- add price column to the orders table
- modify tests to expect order price
- update routes to return order cost as part of response object

[Finishes #161268604]
@akhilome akhilome merged commit fe2ad4a into develop Oct 16, 2018
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 98.182% when pulling 1cc79e5 on ft-order-price-161268604 into c93aed3 on develop.

@akhilome akhilome deleted the ft-order-price-161268604 branch October 18, 2018 13:34
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

2 participants