2023-12-12 Dealita Code Review

 

 

 

 

  1. Ja mēs rakstam kodu KLASĒ tad ir aizliegts rakstīt funkcijas ārpus klases faila, salabot, ka funkcijas atrodās iekšā ControllerDeals

    image-20231212120415668

     

     

     

  1. Jau vairākas reizes esmu brīdinājis - absolūti aizliegts šādi rakstīt bīstamu kodu image-20231212120627750

 

  1. Aizliegts šķirot pēc string mainīgā, ja nomainīs šo objektu, interpeter nesapratīs, ka tas nomainīts - jāizmanto pilns lambda it.modified

image-20231212120723044

 

  1. Aizliegts izmantot untyped datu tipus - aizliegts izmantot dictionaries šādā veidā - jābūt dataclasses ar model klasēm image-20231212120828354

 

  1. Pat pycharm pasvītro, ka tev nav korekta koda struktūra ar metrics - tam ir ne tikai nekorekts datu tips, bet tas nav nodefinēts korektā vietā scope

    image-20231212121016496

 

  1. Pietrūkst typing funkcijām image-20231212121041917

  1. Aizliegts šādi rakstīt funkcijas jābūt vienam return in root scope.

    image-20231212123219771

  2. Pagājušā reizē lūdzu aizpildīt MatchScore logs ar kādu saturu, lai var testēt activities - tas netika izdarīts, sakodēju, lai worker dealiem to aizpilda šobrīd - lai testēti, izmanto jaunus deals, sakodē, ka activities korekti parādās

  3. image-20231113123811518

    image-20231212123531315

  4. Pielabot dizainus kā bija Figma login screen, aizvietot PNG logo ar SVG logo (izeksportēt / nokopēt no Figma ). SVG logo jau ir pieejams projektā image-20231212112240383

 

  1. Uztaisīt mazākus fontus Status un progress saskaņojot ar Raiti, bet Deal title sadalīt vairākās rindās - ja vienam no 3 deals ir 2 rindas, tad arī pārējiem paplašināt height izmēru, kā arī Title samazināt mazliet fonta izmēru

image-20231212120043866

  1. Implementēt delta izmaiņas score no MatchScore logs - ne tikai tur bet visās vietās, kur ir delta izmaiņas (rādīt tieši pēdējās) image-20231212124657606

 

  1. Salabot statistiku Deals sadaļā image-20231212121308031

 

  1. Implementēt Paging Deals sadaļā un CSS stilus progress bars image-20231212121546225

  1. šeit jāielasa checklists un šos uzspiežot jānofiltrē deals tikai ar šo konkrēto checklist image-20231212121639815

 

  1. Finish pogai ir jānoslēdz deal - jānomaina status image-20231212123100574

 

  1. Šeit arī rādīt visus match score logs - visu deals ietvaros image-20231212132441824