search

Framinus / pizza-restaurant

An API built in Node, Express and PostgreSQL
0 stars 0 forks source link

Change pizza_cart and drink_cart #4

Closed Framinus closed 6 years ago

Framinus commented 6 years ago

instead of having these two tables, we can put a cart_id directly on the pizza table.

Framinus commented 6 years ago

I can work on this one! Let me know if this interferes with any schema changes. I'll push up my changes as a branch and not merge it until we figure that out!

Framinus commented 6 years ago

I realized that we can get rid of pizza cart by assigning a cart_id to pizza, but we run into a problem trying to get rid of the drinks_cart table - since we are not assembling a drink, there isn't that extra level of abstraction, and the only other table associated with drink is the base drinks table, and we don't want to assign a cart_id to that. maybe we just leave the drinks_cart table for now...

Framinus commented 6 years ago

Solution is on the issue4 branch.

alexandrawaite commented 6 years ago

I see what you mean about the drink_cart table. I think this solution works great!