search

RIAEvangelist / node-dominos-pizza-api

This is a node.js wrapper for the dominos pizza apis
MIT License
534 stars 129 forks source link

refactor getMenu #32

Closed RIAEvangelist closed 8 years ago

RIAEvangelist commented 8 years ago

need to apply some data massaging so that the Code on Products is the one we send.

perhaps create a separate menu of pizzapi.Item's which can just be added to the order with minimal thought.

RIAEvangelist commented 8 years ago

@julianh2o if you have any ideas we can track the convo here.

julianh2o commented 8 years ago

Our current Item class encapsulates a "shopping cart item" and it combines a product (code), a quantity, and options.. A menu item obviously doesn't include quantity and options.. I'm thinking we may want to implement a new class MenuItem to represent this. Thoughts?

RIAEvangelist commented 8 years ago

seems like this is working