vsvala / runoarkisto Goto Github PK

Ladattu 13.8.2018 00:23

Riippuvuuksien lataamisessa oli ongelmia, minkä vuoksi en päässyt testaamaan sovellusta paikallisesti/virtuaaliympäristössä.

Heroku-sovellus vaikuttaa toimivan (lukuunottamatta "internal server erroreita", mihin en ainakaan itse tiedä, miten vaikuttaa). Sen ja koodinluvun perusteella sanoisin, että sovellus toimii niin kuin tässä vaiheessa pitääkin.

Sovelluksen juuressa oleva "runoarkisto.py" -tiedosto on ilmeisesti käytöstä poistunut testitiedosto, joka olisi hyvä poistaa selkeyden vuoksi.

/application/auth/views.py-tiedostossa tuodaan kahdesti "app" ("from application import app, db" ja "from application import app")

Poiskommentoitu koodi (esim. application/runot/views.py ja application/templates/runot-kansion tiedostoissa one.html, list.html) kannattaa pitää minimissään. Edelliset toteutukset, mikäli pushattu, löytyvät repositorion historiasta.

Suunnittelemasi katerogia-tietokohde tulee olemaan maino lisäys.

Vähän jäi vielä epäselväksi, miten runon suosiota mitattaisiin, mutta jättäisin sen toteuttamisen joka tapauksessa viimeisimmäksi (jos vaan muut kurssin arvosteluperusteet saa täyteen muilla toiminnallisuuksilla).

Dokumentointi on erinomaista: kaikki on selkeästi ja yksityiskohtaisesti selitetty, mm. eri roolien oikeudet ja käyttötapaukset on eritelty hyvin.

Kaiken kaikkiaan sovellus on hyvällä mallilla!

Testattu 27.8. noin 11:30

Käyttäjän rekisteröinti : ilmoittaa runon olevan jo arkistossa, vaikka pitäisi varmaan ilmoittaa käyttäjän olevan jo rekisteröitynyt. Ei kuitenkaan kaadu tarkistukseen, joka on hyvä.

Runon haku nimellä / kategorialla: Olisi parempi, jos esimerkiksi hakusanalla ”onn” haulla löytäisi runon ”Onnellinen”, tai vähintäänkin hakusanalla ”onnellinen”. Helpottaisi hakemista, jos hakusanan ei tarvitsisi olla täsmälleen sama kuin haettava nimi.

Kategorialistauksessa lukee myös kategorian id, turhaa tietoa käyttäjälle?

Runon muokkaus: kategorian voisi lisätä samalla tavalla listasta kuin uuden runon lisäyksessä, jos haluttu kategoria puuttuu niin vasta sitten kirjoittamalla?

Lomakkeet eivät hyväksy liian pitkiä syötteitä, ulkoasu on hyvä ja selkeä, sovellus toimii kaikkiaan hyvin. Lähdekoodi on hyvin kommentoitua ja selkeää. Sovelluksen rakenne on hyvä. Kunhan gitissä listatut tulevat toiminnallisuudet lisätään, työllä on varmasti hyvä mahdollisuus arvosanaan 5. Hyvää työtä!