Koodikatselmointi (13.1 22:15)

Koodi oli selkeästi luettavaa. Mielenkiintoinen idea olla käyttämättä wtformsia ja tehdä lomakkeet sekä syötteiden validaatio itse. Tämä oli mielestäni toteutettu hyvin käyttäen geneeristä luokkaa.

Omalla tehtävälistalla olevien asioiden statuksen muuttaminen (nappi jossa nuoli) tekee post-pyynnön palvelimelle, joka tulkitaan epävalidiksi ja uudelleen ohjaukseen on lisätty http status 400 mikä saa uudelleen ohjauksen, ainaskin omalla selaimella, jumittamaan.

Vertaisarvio projektista ASTIAT

Latasin projektin lauantaina 5.1 klo ~16

Varsinainen palaute

Tähän mennessä toteutettu toiminnallisuus toimii hyvin ja luotettavasti. Käyttöliittymä on kohtalaisen helposti navigoitavvissa, vaikka se kaipaakin selvästi hiomista. Projektin koodi näyttää hyvältä, tai ainakin sen heikkouksien tunnistaminen vaatisi syvempää Python-osaamista kuin minulla on.

Puuttuvia/puutteellisia toiminnallisuuksia sekä paranneusehdotuksia, karkeassa tärkeysjärjestyksessä

Tämä lista ei sisällä käyttötapaukset.rst:ssa listattuja

• Astioita ei voi järjestää tai rajata minkään mukaan
• Astioita ei voi poistaa
• Juuresta voitaisiin suoraan uudelleenohjata astialistaan, minua ei juurikaan kiinnosta että "Hei maailma!"
• Olisi käyttäjäystävällismpää jos astian muutosten tallentaminen palauttaisi näkymän suoraan astialistaan, tai ainakin että olisi vaihtoehtoinen "tallenna & palaa"-nappi
• Kuvaus (ja varmaan muutkin kentät) näytetään aina kokonaan ja yhdellä rivillä vaikka se olisi kuinka pitkä
• Jos yrittää luoda astian jolla ei ole otsikkoa, mutta on kuvaus, mitään ei tapahdu, kuvaus vain katoaa

Typot tms. pilkunviilaus

En tunne Pythonin nimeämiskäytäntöjä hyvin, mutta kuvittelisin että columns.py pitäisi olla column.py, sillä se sisältää vain tämän yhden luokan?