joonakauranen / hy-tsoha-keskustelusovellus Goto Github PK

Sovellus näyttää olevan jo melko hyvässä vaiheessa. Koodi näyttää siistiltä ja muuttujat pääosin hyvin nimettyjä. Tietokantataulut vaikuttaa järkeviltä, ehkä olisi mahdollista yhdistää taulut users ja points?

Parannusehdotuksia, mikäli jää aikaa toteuttaa:

• Uutta käyttäjää luodessa en aluksi ymmärtänyt mikä on lomakkeen kolmas kenttä koska sitä ei mainittu missään? Oletin että se on salasanan varmistamista varten, vasta sisään kirjautuessa selvisi käyttäjäroolit. Tuon voisi jotenkin muulla tavalla toteuttaa, normaalitilanteessa ei varmaan ok että kuka tahansa voi tehdä admin-tilin. Salasanan varmennuksen voisi lisätä
• Sisäänkirjautuminen ei anna virheilmoituksia mikäli yrittää vääriä tunnuksia tai tyhjiä kenttiä, sellaisen voisi lisätä
• Yleisesti navigointia voisi helpottaa, esim. kirjautumis- ja rekisteröitymissivuilta puuttuu etusivu-painikkeet
• Tällä hetkellä käyttäjä voi antaa itselleen niin paljon pisteitä kun haluaa, samaa aihetta voi käyttää pisteenantoon uudelleen ja uudelleen. En ole varma onko tämä tarkoitus vai bugi?
• Myös saman aiheen voi lisätä omiin aiheisiin niin monta kertaa kuin haluaa, ei varmaan tarkoituksenmukaista
• Viestiketjuun viestin yhteyteen voisi lisätä viestin kirjoitusajankohdan ja kirjoittajan id:n / nimimerkin ellei kyseessä ole täysin anonyymi palsta. Silloinkin viestiketjun viestejä on hankala erottaa toisistaan
• Jos haluaa lisää toiminnallisuutta, voisi lisätä esim. aiheiden järjestämisen aakkosjärjestyksen tai luontiajan perusteella

Varmaan osa yllä mainituista on jo to-do-listalla mutta tuossa nyt päällimmäiset asiat mitä sovellusta testatessa nousi mieleen :)

Hy-tsoha-keskustelusovellus

Arvio jätetty 22.11.2022

Sovelluksen aihe ja toteutustyyli vastaa ohjeita. Readme on selkeä. Yritys saada Fly.io käyttöön sekoitti WSL-käyttikseni kokonaan, joten en testannut lokaalisti. Jos saan asennukset kuntoon ennen huomisiltaa, palaan asiaan. Ohjeistus lokaaliin testaukseen oli joka tapauksessa annettu, kiitokset siitä.

Sovellus vaikuttaa olevan hyvässä vaiheessa. Koodi on jo pilkottu moduuleihin. Salasanallisen kirjautumisen pohja on toteutettu. Sovelluksen tietokannassa on toistaiseksi kaksi taulua (users ja topics), joiden välillä ei vielä viiteavaimia - jos siis ovat tässä tarpeen. Jäin miettimään, onko topics = keskusteluketjun aihe vai kokonaisen keskustelualueen otsikko? Varmaan molemmissa tapauksissa voisi olla hyvä tietää, kuka kohteen loi, myös keskustelualueesta, jos esim. ylläpitoon kuuluu useampi henkilö. Vähintään varmaan ainakin käyttäjä - viesti - aihe -viitteet tarvitaan. Sitäkin mietin, olisiko hyödyllistä/tarpeellista rajoittaa role-muuttujaan tallennettavat sallitut vaihtoehdot tietokantapuolella.

Templates-kansiossa on new.html, johon voisi lisätä tarkennuksen, mistä asiasta kyse. Koodissa ei ole salaisuuksia ei ole näkyvillä. Muilta osin ei kommentoitavaa koodista.

Commitit vaikuttavat sopivilta (ts. ei ole kaikkia tiedostoja viety kerrallaan jne, committeja tehty riittävän usein) ja viestit selkeiltä. Ihan pieniä muotoilueroja näkyi eri kertojen välillä.

Toivottavasti Fly.io-ongelmiin löytyy piakkoin kurssilla ratkaisuja. Tsemppiä jatkoon, hyvältä näyttää työ tähän asti. Onnistuneesti aloitettu ydinominaisuuksien toteutuksesta.

Edit: tutkin Githubia samaan aikaan kun sinne tuli päivityksiä, ja esim. tauluja tuli tätä kirjoittaessa nyt jo yksi lisää.