Full Stack projektin katselmointi

[Jakousa]

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?

Kävin selailemassa tuloksia monacon kisoista.

Kokemus

Kesti aika pitkään saada data, mutta muuten siistin näköinen sovellus!

Koodi

Tämä https://github.com/f1-2019-results/backend/blob/master/src/util/asyncRequestHandler.ts taitaa pyrkiä tekemään saman kuin https://www.npmjs.com/package/express-async-errors.

Login toiminnot lienevät kuollutta koodia.

Frontin koodissa on käytetty hyviä käytänteitä. En osaa sanoa miten Vuessa on tapana tehdä asioita sillä Vuea ei yleensä näissä näy, mutta itse koodin laatu on kyllä kelpoa.

En ole täysin varma mitä desktop clientissä tapahtuu, erityisesti https://github.com/f1-2019-results/desktop-client/tree/master/src/binaryParser sisältö ihmetyttää. Mutta tämä on luultavasti vain omaa tietämättömyyttä.

Kokonaisuus

Hieno pieni projekti! Toivottavasti jatkat kehitystä vielä tämänkin jälkeen.