beat-it / 01-jsf

1 stars 1 forks source link

GET /cart #5

Closed ladislavP closed 7 years ago

ladislavP commented 7 years ago

GET /cart a PUT /cart nepoužívajú jednotné JSONs.

GET /cart vracia: payment: null, person: null, billingDetails: null, address: null

Zatiaľ, čo PUT /cart body je: "billingAddress", "address", "person", "deliveryType", "paymentMethod"

Navyše aký je rozdiel medzi address a billingDetails? Address predstavuje adresu dodania a billingDetails sú fakturačné údaje aj s adresou?

Ďakujem.

fpt-nessian commented 7 years ago

Ahoj pridam endpointy na ulozenie Delivery a Payment. PUT /cart je len na akoby finalne vytvorenie objednavky, a v nej idu payment, billingDetails a adress atd.

Veci sa pridavaju do kosika cez POST cart/items.

BillingDetails je adresa + company.

Pripadne ak mas na mysli nejake vhodnejsie riesenie, napis, skusim tam nahodit.

ladislavP commented 7 years ago

Neviem za mna by asi bolo lepsie mat len jeden endpoint PUT /cart na ktoru mozem zasla t parcialnu zmenu kosika po prvom kroku delivery a payment po druhom data tykajuce sa osoby a adries. Finalne dkoncenie objednavke je v zasade aspon teraz nepotrebne. Toto riesenie som popisal aj v dokumentacii.

ladislavP commented 7 years ago

Myslim ze toto mozeme uzavriet popis som presunul spolu do tejto chyby: https://github.com/beat-it/01-jsf/issues/10