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

Added Topping Support and Fixed Menu Error #7

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

JPadley18
Copy link
Contributor

@JPadley18 JPadley18 commented Feb 8, 2019

Added support for adding/removing pizza toppings. Also fixed a code problem where the menu would only parse the Sides category from the domino's API. It now iterates over all categories.

Added the ability to see the basket's total price
Menu class now parses more than one subcategory
Added support for additional toppings
Added a full example of adding toppings in the README
@JPadley18 JPadley18 changed the title Fixed Menu Error Added Topping Support and Fixed Menu Error Feb 9, 2019
Added support for removing certain toppings
Added demo of removing certain toppings in README
Example for adding ingredients now appears in README
Added support for IngredientLists to allow for searching by name
Added Pizza and IngredientList classes to improve support for modifying ingredients.
Updated the usage of ingredients to include ingredient list functions.
Shortened file to 250 lines to comply with CodeClimate guidelines
IngredientList now passes Travis-CI Python 2 tests
Repetitive code in IngredientList removed
Updated VARIANT reference in README
Fixed 'Variant' Spelling Error
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.

1 participant