Code review 1: Tietokantasovellus-harjoitustyö

[ruusukivi]

Kunnianhimoinen ja laaja aihe! 💯

Kommentit sovelluksesta Herokussa

• Toimii hyvin: ryhmän lisääminen, threadin lisääminen ja haku
• Ei toimi vielä: threadin äänestäminen, kommentin lisääminen threadiin tai ryhmään liittyminen (yhdessä threadissa oli kommentteja, mutta en itse onnistunut niitä lisäämään)
• Syötteiden pituudet tarkistetaan 👍🏻
• Hakukentän ohje-teksti englanniksi vaikka sovellus muuten suomeksi
• En päässyt urleja manipuloimalla näkemään mitään ei-sallittua 👍🏻
• Huomio: jos lisää saman nimisen ryhmän ei tule virheilmoitusta, mutta ryhmä ei myöskään näy tuplana - ehkä näin tarkoituskin?
• Jos aikaa jää, virhesivut voisi lisätä esim. osoitteille joita ei löydy

Huomiona seuraaviin kommentteihin: tämä on ensimmäinen kurssi, jolla käytän Pythonia, joten sen osalta en osaa kovin syvällisiä kommentoida..

Sovelluksen koodi ja rakenne

• Venv, pycache yms. hakemistot kannattaa varmaan deletoida githubista, olivat ehtineet uida mukaan alussa. Nyt gitignore kunnossa.
• Sovelluksen rakenne vaikutti selkeältä - jaettu järkevästi nimettyihin hakemistoihin 👍🏻
• Lomakkeissa huomioitu syötteiden maksimipituudet - hyvä! 👍🏻
• Csrf huomioitu lomakkeissa - hyvä! 👍🏻 (Lukuunottamatta new.html, mutta se ei tainnut olla käytössä)
• Pitkät SQL-kyselyt kannattaisi ehkä rivittää luettavuuden helpottamiseksi (oliko projektissa pylint käytössä?)

Muuta

• flash() oli minulle uusi tuttavuus- kiitos opin uutta!

[ToxPuro]

Nyt pitäisi rekisteröityminen onnistua