vertaisarviointi2

[mojord]

Rakenne on toimiva ja sovelluksessa oli helppo navigoida. Keskityn tässä olennaisimpiin huomaamiini kohtiin, jotta arviosta olisi toivottavasti hyötyäkin.

• Rekisteröinti onnistuu tyhjällä salasanalla, sovelluksella on nyt käyttäjä "rekku", jolla ei ole salasanaa. Tätä varten voisi lisätä tsekkauksen salasanan ominaisuuksiin rekisteröinnissä (nyt tarkistetaan vain, että salasanat ovat samat).
• Arvioiden listauksessa restaurant.html-sivu nykäisee seuraavan arvion antaneen kiinni edelliseen arvioon, jolloin arvio visuaalisesti kiinnittyy väärään antajaan - Havainnollistin tätä lisäämäni Mustin murkina -ravintolan arvioilla, joissa näyttää, että "Hau, hau mums" arvion olisi antanut miuku (po. rekku) ja "Miuaurrr!"-arvion miuku (po. vaakku) jne. Arvelisin syyksi html-sivun p-tageja.
• Haku ei toimi. Näkyisi olevan routes.py:n def resultissa vain method GET (ja index.html -lomakkeessa), joten kyselylle ei tule payloadia; eli voi fiksata tekemällä register-toiminnon tyyliin.

Muuta:

• Tyhjästä arviosta tulee "Bad request", eli siihen voisi lisätä virheen.
• Kellonaika näkyi ainakin minulla kahdella eri koneella 2 tuntia jäljessä (eli GMT:nä). Täällä (https://www.postgresqltutorial.com/postgresql-now/) näytti olevan hyviä ohjeita ajan konffaukseen, jos ehtii

[qusba]

Hei ja kiitos hyvästä arviosta! Haku itseasiassa toimii, mutta ei ehkä kuvittelemallasi tavalla. Sovelluksen README:ssä lukee, että käyttäjä voi hakea ravintolan 'kuvauksessa' esiintyvän sanan perusteella, eli jos olet yrittänyt hakea esimerkiksi ravintolan nimellä, se ei tuota tulosta. Ymmärrettävästi hieman hassua, mutta toimii kuten on tarkoitettu. Kaikki muut huomautukset olivat todella hyviä ja pyrin niitä korjaamaan!

[mojord]

Totta joo. Jäikin mielikuva, että ensimmäisellä kerralla viime viikolla haku toimi, joten ihmettelin asiaa vähän itsekin. Ilmeisesti aluksi osasin noudattaa ohjetta ja hakea kuvauksen perusteella (minkä takia olin varmaan yhden kuvauksen lisännytkin).