search

zebraf1 / Kassa

Private inventory system
GNU General Public License v2.0
0 stars 0 forks source link

Transfers #85

Closed zebraf1 closed 6 years ago

zebraf1 commented 6 years ago

Transfers muudatused

zebraf1 commented 6 years ago

Üldiselt väga hea töö! Doc olemas ja vajalikud klassid ka. Ainuke, mis puudu on automaattestid: APIBundle/Tests/Controller/TransfersControllerTest.php Vaata teisi näited, väga keeruline pole lisada listi jaoks ja create jaoks mõni lihtne test.

zebraf1 commented 6 years ago

kassa-test panin muudatuse üles, aga sisse logides tuleb viga: Undefined class constant 'TYPE_CASH_PAYMENT'

Sa kaotasid ära TYPE_CASH_PAYMENT ja TYPE_CREDIT_PAYMENT Transactions klassist, aga koodis jäi neid kasutuskohti veel alles. Miks sa need kaotasid üldse? TYPE_CASH_PAYMENT - ok ilmselt seda kasutuslugu väga ei ole, krediiti saab siis ülekandega lisada? (Kasutaja paneb sularaha kassasse ja saab krediiti juurde) TYPE_CREDIT_PAYMENT - see on selleks, et majanduseestseisja saab krediiti kellelegi lisada. Kas see võimalus kaob ära? Sul oli krediidi lisamiseks mingi automaatne võimalus?

zebraf1 commented 6 years ago

CreditManagementController - adjustAction(): siin saab krediiti lisada juurde. Kas see action kaob siis üldse ära? Vaade on ka sellel: https://kassa-test.rotalia.ee/app_debug.php/krediit/ Siin on see krediidi lisamise nupp juba kadunud ma vaatan? Javascript, controlleri kood PurchaseControlleris on kõik endiselt alles. Kui see funktsionaalsus kaob ära, siis peab kõik koodi puhastama.

tanelk commented 6 years ago

Selle vana UI poole pealt pole mina muudatusi teinud, ehk siis ühtegi nuppu ma küll ära ei kaotanud.

Need tüübid läksid mul tõesti kaduma, see polnud tahtlik.

Praegu ma tegin nii, et krediidi lisamine vajab admini õiguseid. Minu jaoks on kogu selle projekti suur eesmärk saada konvendist sularaha minema (sellega on alailma jamasi). Ehk siis kasutjal sularaha krediidi vastu vahetamine töötaks sellele vastu. Automaatika visioonist ma #83 all rääkisin aga see pole esimene prioriteet.

tanelk commented 6 years ago

Võtsin Transactions klassi muudatused tagasi.

Avastasin ühe huvitava koha - sa oled vahepeal muutnud klassi ProductType nii, et GET result annab välja parameetri "productGroupId" aga POST ja PATCH tahavad "productGroup", kuigi mõlemad viitavad id-le.

Kas see oli tahtlik muudatus? Mul oleks UI poole pealt märksa mugavam, kui mõlemat pidi oleks samad nimed (eelistatult id oleks id lõpuga).

Hetkel ei saa selle pärast tooteid lisada/muuta.

zebraf1 commented 6 years ago

Okei, ma ajan vana UI ise korda. Mõtlesingi, et kas see oli tahtlik eemaldamine või kogemata 🐛

zebraf1 commented 6 years ago

A sa juba panid tagasi need asjad, kassa-test hakkas nende uuenduste peale tööle

zebraf1 commented 6 years ago

Ahsaa, ma muutsin seda ProductType klassi jah, meie majanduseestseisjal oli probleeme, et toote muutmine oli katki läinud. Millegipärast productGroupId'ga ei toiminud see model tüüpi väli, andis viga, kuna submittimisel tekkis "Õlu" väärtuseks, mitte 1. API puhul peaks olemas see ID lõpuga jah... Teen #86 raames korda, seal läks katki