Skip to content

Vertaisarviointi #1

@kerkkanen

Description

@kerkkanen

Heipähei!

Sovellus toimi Herokussa hienosti ja näytti hyvältä. Tässäpä muutama ajatus käytettävyydestä ja hyvin vähän muusta.

Käytettävyys

  • Tuntui mukavalta käyttää keskeneräistäkin sovellusta, kun ulkoasu oli laitettu heti hieman muotoillumpaan muotoon.
  • Toimintoja ei vielä ole paljon, mutta olemassaolevat tuntuvat toimiva kuten pitääkin.
  • Nämä ovat varmasti makuasioita, mutta lausuttakoon ääneen ja jätetään tekijän harkintaan:
    • Tykkäsin muutoin ulkoasun väreistä, mutta (ainakin omalla näyöllä) harmaa teksti oli hieman vaikeasti luettavaa.
    • Kirjautumisen virheviestit avasivat erillisen virhesivun. Näin oli taidettu tehdä referenssisovelluksessakin, mutta käyttökokemusta voisi tehdä mukavammaksi, jos esim. virheellinen salasana -ilmoitus tulisi näkyviin kirjautumissivulle eikä tarvitsisi klikkailla takaisin-nappulaa erikseen päästäkseen yrittämään kirjautumista uudelleen.
    • Erityisesti rekisteröityessä joutuu palaamaan aina etusivulle ja sieltä klikkaamaan uudelleen rekisteröitymissivulle, jos salasana ei tai tunnus ei ole kelvollinen (salasanan ehdot voisi kenties myös ilmaista yhdessä virheviestissä).

Koodi

  • Koodi on todella siistiä ja helppolukuista.
  • Toimintoja ei ollut vielä valtavasti, mutta esim. kirjautuminen ja käyttäjäksi rekisteröityminen näytti myös koodin puolesta hyvältä.
  • Tietoturva kunnossa! Ei haavoittuvuuksia SQL-kutsuissa, csrf-tokenia käytetty ja oikeat tiedostot salattu.

Minun silmääni työ näyttäisi olevan erinomaisella alulla, joten korjaus- tai parannusehdotukset tässä vaiheessa jäävät aika lyhyeksi listaksi. Ulkonäköä on jo mietitty, rakenne tuntuu järkevältä ja koodi on siistiä ja selkeää. Sovelluksen idea kuulostaa myös oikeasti hyödylliseltä. ja aion ehdottomasti tulla vilkaisemaan sitä, kun toiminnallisuuksia on valmistunut enemmän.

Kiitos työstä j tsemppiä pakertamiseen!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions