search

beat-it / 01-jsf

1 stars 1 forks source link

Inkonzistencia dat API #10

Open ladislavP opened 7 years ago

ladislavP commented 7 years ago

Ahojte. Toto si myslim ze by sme mali odldadit aby sme mohli upravit dokumentaciu pre ludi, ktori by si to chceli rozbehat aby k tomu mali nejaky podklad. :)

  1. Mohli by sme nejako zjednotiť to API aby GET a PUT metódy nemali rozlišné JSON štruktúry. (Môžeme ak chcete prejsť osobne zajtra). V PUT-E máme billingAddress v GET-e máme billingDetails.

  2. V pripade 1. requestu GET /cart dostanem {address: null} avšak keď chcem zrušiť odlišnú dodaciu adresu, a urobim PUT /cart s telickom {address: null} začne GET /cart vraciať {address: {street: "", ...}}. Predpokladam ze podobne to bude aj s company.

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.

pdudits commented 7 years ago

billingDetails má billingAddress a company. Takže aj PUT by mal mať billingDetails. A vlastne aj má, iba sa to zle volá v CartOrderDTO. Ako ja osobne až tak ďaleko nezájdem aby som to POSToval, lebo aj tak nebude čas to vysvetliť.