Full Stack projektin katselmointi

[Kaltsoon]

Katselmointi

Terve @Otju! Tässä lyhyt katselmointi Full Stack kurssin projektistasi.

Käytettävyys

Mitä tein?

• Selasin juomia eli hakuehdoilla
• Vertailin juomia
• Testasin annoslaskuria
• Katsoin tilastoja

Kokemus

• Sovelluksella on tyylikäs helppokäyttöinen käyttöliittymä, hyvää työtä! 🏆
• Hieno logo!
• Toiminallisuus on toteuttu hyvin. Ei bugeja, eikä erityisiä käytettävyysongelmia

Koodi

• Frontend-puolen koodi on selkeästi organisoitua, hyvä!
• Hyviä hook-abstraktioita toteutettu
• GraphQL-kyselyissä voisi hyödyntää fragmentteja, ettei kentistä tulisi copy pastea
• Inline-tyylejä kannattaa välttää
• Kuvien poluissa olisi helpompi käyttää Webpackin file-loaderia, joka on CRA:ssakin käytössä
• Myös backend-puolen koodi on selkeästi organisoitua, hienoa!
• GraphQL-skeeman koodi on hajautettu selkeästi
• Populoinnin sijaan kannattaa käyttää dataloaderia ja resolvereita kentissä
• Scaraper-koodi on toteuttu järkevästi niin, että uusien scaperien lisääminen vaikuttaa olevan kohtalaisen vaivatonta

Kokonaisuus

Mainio sovellus, jossa tyylikäs ja käyttäjäystävällinen käyttöliittymä. Sovelluksella on selkeä arkkitehtuuri, palveluiden vastuutehtävät on eritelty selkeästi. Koodi on kaikilta osin järkevästi organisoitua, eikä sen laadusta löytynyt pienten refaktorointiehdotusten lisäksi erityistä moitittavaa.