MiikaM
commented
3 years ago Kiitos, loistavaa! Joo täytyy ottaa huomioon näitä ehotuksia seuraavissa projekteissa. Tämä projekti olikin ensimmäinen isompi verkkosivu niin välillä jäi huomaamatta simppeleitä asioita kuten nuo rivinvaihdot, vaikka eslint olikin käytössä. 👍🏼
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
Kirjauduin sisään
Kommentoin kiipeilypaikkaa
Kokemus
Erittäin hienot sivut.
Oli virkistävää joutua varmentamaan sähköpostiosoitteen näissä projekteissa. Ja sekin toimi hyvin.
Pientä tekstin hyppimistä etusivulla olevien ruutujen kasvun ja pienenmisen myötä. Ja on hieman hassua että kaikki sisältö pompahtaa ruudulle. Muuten näyttää hyvältä.
Koodi
Erittäin hyvin jaettu koodi useaan palaseen. Tämähän on lähes timanttista.
git on parempi versionhallinnan työkalu kuin kommentit https://github.com/MiikaM/ClimbingFinland/blob/b16c0048c2e986ee3eecd476d807eec49978050b/frontend/src/components/Places/info.js
Huomasin että backend koodissa voisi olla iloa tällaisesta: https://www.npmjs.com/package/express-async-errors
https://github.com/MiikaM/ClimbingFinland/blob/b16c0048c2e986ee3eecd476d807eec49978050b/backend/controllers/places.js
Pääset eroon copy paste try catcheistä ja voit käsitellä virheet suoraan middlewaressa.
Sinulla on muutamassa kohtaa erikoisesti rivinvaihtoja, käytätkö eslint tai prettier kirjastoja apuna? Voisit harkita niitä.
Kokonaisuus
Esimerkillinen projekti! Hienoa työtä.