search

Elbeg0406 / Project-3

0 stars 0 forks source link

Code review #3

Open biligunb opened 3 years ago

biligunb commented 3 years ago
Elbeg0406 commented 3 years ago

<--------- Research coding principles ------------> Эмх цэгцтэй ойлгомжтой бичих талаар зөвлөгөөг өөртөө хэрэгжүүлийх хичээнэ. Format -ийн хувьд VS Code -д байдаг prettier code extension ашиглаж байгаа

Elbeg0406 commented 3 years ago

<<<<<<< Research code review >>>>>>>> Код хянах: Бурхан биш учраас алдаатай код хийнэ. Тэрийг засварлах, дүн шинжилгээ хийх, илүү сайжруулах, ойлгомжтой болгох үүднээс 2-оос дээш хүн хянах үйл ажиллагаа гэж ойлголоо.

Elbeg0406 commented 3 years ago

<<<<<<<< Research pair programming >>>>>>>>> Хос програмчлал: Код хянах нь алдааг засаж сайжруулахын тулд байсан бол Хос програмчлалын ойлголт нь анхнаасаа л 2уулаа хий алдаа бага гарна. Гэхдээ 2уулаа зэрэг код бичээд бай гэсэн үг биш. Нэг нь бичиж нөгөө нь хянаж сайжруулалт талаас нь анхаарах, чиглүүлэх үүрэгтэй.

Elbeg0406 commented 3 years ago

<<<<<<<<< Try pair programming >>>>>>>>> Багийн ажиллагаа. Бид 2 хийгээд байгаа project pair programming. Багаар ажиллах чадварт суралцах хэрэгтэй. Хүнтэй харилцах чадвар чухал. Юу хийж байгаа, юу болохгүй байгаа, цаашид юу хийх талаар нээлттэй, оновчтой асуулт асууж, тулгамдаж буй асуудалд шийдвэр гаргалцах.

Elbeg0406 commented 3 years ago

<<<<<<<<<< Try code review >>>>>>>>> Эцсийн бүтээгдэхүүн чанартай байх ёстой тул кодыг заавал хянах ёстой. Мэдээж хүний бичсэн кодыг өөр хүн хараад ойлгогдохоор зөв бичэглэлтэй, нэршилийн хувьд оновчтой ойлгогдохоор байх хэрэгтэй.

biligunb commented 3 years ago

<<<<<<<<< Try pair programming >>>>>>>>> Багийн ажиллагаа. Бид 2 хийгээд байгаа project pair programming. Багаар ажиллах чадварт суралцах хэрэгтэй. Хүнтэй харилцах чадвар чухал. Юу хийж байгаа, юу болохгүй байгаа, цаашид юу хийх талаар нээлттэй, оновчтой асуулт асууж, тулгамдаж буй асуудалд шийдвэр гаргалцах.

Pair gedeg ni gehdee buur yag delgetsee share-leed hamtdaa sugaad hiih ni shuu.

Elbeg0406 commented 3 years ago

<<<<<<<<<<<<<<< Add date field & nuutsUg field to TODO >>>>>>>>>>>>>> ToDo List нэмдэг хэсэгт date, password field нэмсэн. Нууц үг шалгадаг хэсэг дутуу байна todo

Elbeg0406 commented 3 years ago

<<<<<<<<<< Add validation to all fields >>>>>>>>>>>>>> Date must be greater than NOW - Date() функцээр одоогийн огноог авч байгаа title less than 60 characters - 60 тэмдэгтийн урттайгаар байгуулсан entity

Elbeg0406 commented 3 years ago

Password гараас өгхүү, databse дээр шууд insert хийгээд өгсөн байгаа

https://user-images.githubusercontent.com/62293207/116237579-fe649300-a792-11eb-969d-ea92f404d02c.mp4

Энэ чиний өгсөн шаардлагаас өөр байгааз, тэгээд material-ui дээр validation нь ингэж авхаас өөр юм байдаггүй яахын бүү мэд, ухаад л байна... Screenshot from 2021-04-27 20-11-10

Elbeg0406 commented 3 years ago

<<<<<<<< Add validation to all fields >>>>>>>>>> todo

pass

Elbeg0406 commented 3 years ago

<<<<<<<<< Update #2 test code accordingly >>>>>>>>>>>>> Асуудал: Insomnia дээр Validation ажиллаад, тест код дээр ажиллахгүй байгаамаа

Асуудал: Бодсон шиг биш юм байна, UsePipes дуудахгүй шууд функц дуудаад явчлаа, за дахиад юм ухахаас, TestModule дотроо ValidationPipe зааж өгхийн болвуу гэж бодном