CaseEight
commented
6 months ago - README käyttöohjetta voisi tarkentaa; ks. vertaisarviointi ohjeesta kohdat tietokannoista.
- HTML sivu näyttää hyvältä, sovelluksen sivut selkeästi näkyvissä yläpalkissa
- Käyttäjätunnuksen luonti onnistuu, salasanan toistaminen hyvä ajatus.
- Omat tehtävät (/home) ei vielä toiminnassa, painike vie etusivulle.
- Vaikka olen kirjautunut sisään, etusivulla lukee että en ole kirjautunut sisään.
- Kuin koitan kirjata käyttäjän ulos -Internal Server Error (KeyError: 'user_id' terminaalissa)
- Kielen vaihtamisesta ei tapahdu mitään
- Omat tehtävät painike vie etusivulle
- Käsin kirjoittamalla osoiteriville /new pääsin kuitenkin luomaan uuden tehtävän
- Tehtävän luonti onnistuu
- Pakolliset kentät ja validin päivämäärän tarkastus hyvä
- Voisi kehittää niin, että päivämäärä ei voi olla menneisyydessä?
- /delete sivu aukeaa myös käsin, mutta siinä ei näy poistettavia muistiinpanoja
- Koodi on jaettu useampaan .py-tiedostoon, joka pitää koodin selkeänä.
- Routes.py; hyvä tapasi käsitellä lähettää spesifit virheviestit eri tapauksista.
- Schemassa vasta kaksi taulukkoa tietojen käsittelyyn. Luultavasti tulee yksi lisää tehtävälistoille. Ja mielestäni muistutukset/toistuvuudet kannattanee määritellä myös omaksi taulukokseen. Ohjearvo kurssin sovelluksille on 5-10 taulukon käyttäminen.
- Mielestäni oikein hyvällä mallilla jo tässä kohtaa :)
veeranniina