Vertaisarviointi 1

[ronituohino]

Hei @sari-bee! Testailin sovellustasi tällä viikolla ja tässä on katsaus havainnoistani.

Kätettävyys

Mitä tein?

• Avasin Herokussa olevan sovelluksen
• Kokeilin Verivaraston hakutoimintoja netissä
• Kloonasin projektin omalle koneelleni ja käynnistin sen paikallisesti
• Lisäsin uuden verivaraston, hoitoyksikön ja valmistetyypin ylläpitotoiminnoista
• Sisäänkirjasin verivalmisteita
• Loin uuden potilaan
• Kirjasin verensiirron

Kokemus

• Sovelluksen käyttöliittymä on hyvin tyylikäs ja moderni!
• Lomakkeet ovat intuitiiviset ja helppokäyttöiset
  • Validoinnit puuttuvat, mutta mainitsitkin jo tästä katsauksessa
  • Korostan myös, että tyhjän valinnan submitointi johtaa virheeseen
  • Olisi kiva, jos lomakkeiden napit muuttuisivat ns. disabled -tilaan jos syöte on puutteellinen
• Olisi myös hyvä, jos sovelluksen README -tiedostossa olisi hieman dokumentaatiota sovelluksen paikalliseen käynnistykseen
  • Myös .env -muuttujien dokumentaatio

Koodi

• Backend -koodi on selkeästi organisoitu, hyvä!
  • Ohjelmointityyli seuraa Pythonin tyyliohjetta, hyvä!
• Frontend -koodi näyttää hyvältä
  • Suosittelen käyttämään erillisiä CSS -tiedostoja tyylien määrittelyssä
  • Suosittelen myös käyttämään hyväksi Flask -kirjaston extends -komentoa
• Tietokannat on suunniteltu järkevästi
  • SQL -toiminnallisuuksia on käytetty hyväksi kyselyissä
• Kokonaisuudessaan koodi vaikuttaa yhteinäiseltä ja selkeältä, erinomaista!

Muuta

• Arvosteluperusteiden perusvaatimukset mielestäni täyttyvät
• Versionhallinta näyttää hyvältä, vain vaadittavat tiedostot, hyvät commit -viestit
• Tietoturvaa on ehkä parempi arvioida sitten kun käyttäjäjärjestelmä on toteutettu

Kokonaisuus

Kokonaisuudessaan projekti on tällä hetkellä erinomaisessa vaiheessa. Sovellusta on miellyttävä käyttää ja toiminnallisuus on toteutettu hyvin. Hyvää työtä!

[sari-bee]

Hei, kiitos paljon kommenteistasi ja hyvistä ehdotuksistasi, huomioin ne sovelluksen jatkokehityksessä!