Katselmointi

[Jakousa]

Katselmointi

Sivuston käytettävyys

Mitä tein

• 1.12.2019 klo ~12:30

• Kirjauduin sisään

  • Klikkailin about kaikkialle, en kuitenkaan poistanut mitään

Kokemus

• Käyttäjänimestä / salasanasta voisi trimmata whitespacen. Vahingossa copypastesin kenttiin jotain huonosti

• Mahtavaa työtä!

• Kirjautumissivulle voisi laittaa ohjeistuksen "Mikä tämä on" jos sinne vaikka joku eksyy

• Navbarissa valkoinen piste oikeassa ja vasemmassa yläreunassa.

Koodi

• Koodi löytyy juuri sieltä mistä pitää, erittäin hyvin nimetyt tiedostot ja kansiot

• Harkitse https://www.npmjs.com/package/express-async-errors käyttöönottoa ettei tarvitse try catchiä laittaa kaikkialle.

• Tokenin validointiin suosittelen middlewarea. Sillä saisi vielä enemmän siistittyä controllereita.

  • Jos tarvitset tokenin käyttäjää niin middlewareen on helppo laittaa jotakuinkin näin (pseudokoodia):

    req.currentUser = getUserFromToken(decodedToken)
  

  tämän jälkeen käyttäjä on controllerissa aina saatavilla req.currentUserista

• Frontin koodia hiottu selkeästi enemmän. Hienoa!

Kokonaisuus

• Projektista huomaa että se on tehty juuri käyttöä varten ja aikaa on käytetty myös frontin ja backin ulkopuolella miettimään ratkaisuja. Hyvää työtä!

• Sain sellaisen kuvan työstä että se on jo käytössä. Hyvä!

  • Viettäisin hetken aikaa backendin kanssa varmistaessa että sen ylläpidettävyys paranee, mutta tästä on hyvä jatkaa!

🎵 / 🎼

PS. Korjaa tämä:

Nyt on conffit jotenkin ristissä. Lähde selvittelemään käyttämällä esim
https://superuser.com/questions/1435213/github-why-do-i-appear-twice-on-every-commit

ja

https://help.github.com/en/github/setting-up-and-managing-your-github-user-account/setting-your-commit-email-address