2023-12-28 Dealita Code Review
Code review
Dažas lietas pats salaboju, bet pārbaudi, kā vel nav kaut kur šādi gļuki. Lai šo izdarītu loģiski, ka vajadzēja veikt JOIN
Ja atgriež LIST type aizliegts atgriezt None - jāatgriež tukšs list by default
Aizliegts Kopēt Due Dilligence tabulu kodu starp Deal un Due Dilligence sadaļām ir jāizmanto Flask template Macro!! https://github.com/asya-ai/dealita-web/issues/26
Kāda iemesla dēļ tu 2x pievienoji vienu un to pašu kodu, kuru jau es biju pievienojis??
Kāpēc šie ir dictionaries nevis LISTS????
kas notiks ar lapu, ja es padošu
?page=100kāpēc nav nekādu checks?
Absolūti aizliegts šādi rakstīt kodu! šeit ir jāveic SQL sorting un limit NEVIS python pusē ar visu tabulu! Pie tam netiek veikti nekādi filtri - visu useru deals šobrīd tiek rādīti!
Pie tam tev rekur bija tieši jau paraugs, ka ir jāizmanto user_company_id
Pagging NEKAD NEKAD šādi nedara, tāpēc arī tā lapa tik lēni strādā
To dara šādi ar SQL
Nedrīkst N reizes atkārtot tik dārgu SQL SELECT !!
Aizliegts izmantot šādus magic dict
10.
Un kas notiks, ja būs jaunam lietotājam tikai viens deal? kas tas par dīvainu IF statment?? Vai template pusē tas arī ir apstrādāts?
AIZLIEGTS pievienot data classes neesošus properites - tād neeksistē DB
CSS un code review
TODO
Kad salabo zemāk dotos uzdevumus GIT dari sekojošas darbības pie GIT issue:
Veicot izmaiņas kodā atsaukties ar hash tag uz issue, kurš tika salabots konkrētajā GIT commit!
Nepieciešams steidzami salabot šīs issues: