2024-04-29 Code Review Dealita

  1. Lai apturētu, iesartētu workers, izmanto screen commands uz severa

https://share.yellowrobot.xyz/quick/2024-4-29-F74B6AED-EE7D-4A80-AFDF-2FAAECC6451C.html

 

  1. Pārbaudīt vai jauniem darījumiem, tiešām šeit nav None, ja ir atrast iemeslu un salabot

image-20240429181201842

  1. šeit bija pilnīgi otrādi loģika uzkodēta - QUESTIONS pārbauda faktos! Kad taisi jaunu funkciju, kas strādā ar trūkstošiem faktiem, lūdzu izveido visu lielo process_deal funkciju jaunā koda daļā, lai atkal nesalauztu strādājošu kodu

image-20240429181751998

 

  1. Nedrīkst būt hardcoded strings, izmantot caur translations

image-20240429182915529

 

 

  1. Pievienoju kodu ar kuru tu vari debuggot pa taisno palaižot ControllerLLM failu kā main code

    image-20240429220907015

 

  1. Ja critical, tad nesūtīt epastus pat ja varētu max_emails un pietrūkst info

image-20240429183054421

 

image-20240429183218211

 

  1. Nepareiza loģika un send_email nesaprotama funkcija

image-20240429183401553

 

image-20240429183529700

image-20240429183748970

 

  1. Salabot un notestēt epasta faktu splitting funkciju

image-20240429183919381

 

image-20240429184254910

 

  1. Aizliegts šādi veidot funkcijas bez typing un ar return/non-return structure

image-20240429184910587

 

 

  1. Jāpārtaisa loģika - ja var sūtīt epastus un ir trūkstoši non-critical jautājumi, tad nepievienot is_match=false answer, bet atstāt tās in-progress, kamēr saņem epastus

image-20240429185636659