Full Stack projektin katselmointi

Tässä lyhyt katselmointi Full Stack kurssin projektistasi. Katselmoinnissa esitetyt kommentit ovat parannusehdotuksia, joita voit halutessasi ottaa huomioon tässä tai tulevissa projekteissasi. Mitään muutoksia tähän projektiin ei siis ole pakko tehdä suoritusmerkintää varten.

Käytettävyys

Mitä tein?

Rekisteröidyin

Lisäsin "viinin"

Arvostelin "viiniä"

Kokemus

Kiva että kirjautui sisään suoraan rekisteröinnistä.

Rekisteröityessä kentät tummia ja teksti tummaa. Voisi vaihtaa väriä johonkin vähän vaaleampaan. Samaa myös muualla..

Todella hyvä hakutoiminnallisuus ja koko äppi selkeä käyttää.

Kehitysideaksi tuli mieleen että on varmasti olemassa jokin tapa (Rajapinta?) saada sisään mahdollisimman monia viinejä jo valmiiksi. Ehkä jopa lisätä kuvia viinin yhteyteen.

Koodi

Komponentit löytyvät juuri mistä odotin. Projekti on myös jaettu hyvän kokosiin palasiin.

Frontin koodista vaikea löytää mitään kehitysideoita. Jotkin komponentit ehkä vähän pitkiä ja https://github.com/jkseppal/Winebook/blob/f11c5b77d178b5f6112abe11af8f6ac744dfc594/frontend/src/components/WineForm.js#L121-L135 voisi vaihtaa listan läpikäyntiin copy-pasten sijaan.

Bäkissä https://github.com/jkseppal/Winebook/blob/f11c5b77d178b5f6112abe11af8f6ac744dfc594/backend/controllers/blogs.js#L16-L23 koodin voisi siirtää middlewareen. Esim jos talletat käyttäjän request.currentUser on käyttäjä saatavilla kontrollerissa suoraan request.currentUserista sen sijaan että tarvitsisi useassa paikassa tarkistaa tokenia ja käyttäjää.

Kokonaisuus

Vaikka ulkoasuun toki voisi hieman vielä panostaa tämä on erittäin hyvälaatuinen projekti ja tätä kelpaa pienen refaktoroinnin jälkeen jatkokehittää!

jkseppal / Winebook