-
Notifications
You must be signed in to change notification settings - Fork 1
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
Ike complete order database refactor #33
Conversation
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.
you the main banana 🍌
tests pass!
MAIN BANANAS 🍌 🍌 🍌 |
works amazing!!!! RUNS ALL THE TESTS 🥇 👍 it's bananas 🍌 |
|
||
|
||
|
||
# except: |
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.
Please remove commented code before submitting a PR.
@@ -47,6 +47,7 @@ def add_product_to_order(self, product): | |||
AND quantity = {} | |||
""".format(product.get_product_title(), product.get_product_description(), product.get_product_price(), product.get_product_quantity())) | |||
selected_product = cursor.fetchone() | |||
print(selected_product, "this is selected prod w zoe") |
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.
Remove debugging statements before submitting a PR
selected_product = cursor.fetchone() | ||
cursor.close() | ||
|
||
rfk = Customer("Robert Kennedy", '1919 Heron Pointe Dr', "Nashville", "TN", 37214, '8888978880') |
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.
Some white space to break up the code into conceptual blocks, along with comments, will make this code easier to read
Description
Complete an order by tying payment type to order and setting payment complete to 1.
Number of Fixes
#6 , #9, #20,
Related Ticket(s)
#1 ,
Problem to Solve
Describe the problem(s) this pull request solves.
Tests a customer can complete an order e.g.-Allows a customer to complete an order (add payment type to an order and set order complete to 1)
Proposed Changes
Describe the proposed changes.
Expected Behavior
Describe the expected behavior.
Within the database, selected order should be 1
Steps to Test Solution
run python runtests.py
Testing
[x ] There are new unit tests in this PR, and I verify that there is full coverage of all new code.
[ x] I certify that all existing tests pass
Documentation
[ x] There is new documentation in this pull request that must be reviewed..
[ x] I added documentation for any new classes/methods
Deploy Notes
Notes regarding deployment the contained body of work. These should note any
db migrations, etc.