Skip to content

Fullstack-harjoitustyön katselmointi #5

@ShootingStar91

Description

@ShootingStar91

Fullstack-harjoitustyön katselmointi

Tämä on katselmointi fullstack-harjoitusprojektistanne. Sinun ei tarvitse toteuttaa tässä mainittuja parannusehdotuksia saadaksesi suoritusmerkinnät.

Käyttökokemus

  • Sovellus on hyvin tyylikäs ja selkeä ulkoasultaan
  • Responsiivisuus toimii todella hyvin
  • Notifikaatiot hoidettu tyylikkäästi, validaatiot löytyy ja niistä tulee selkeä ilmoitus
  • En aluksi meinannut löytää mistä luoda uusi Note. Navbar on yleensä varattu sivuston näkymien välillä navigointiin, uuden Noten lisääminen on minusta avaintoiminto jonka tulisi löytyä note-näkymästä selkeämmin. Plus-merkki on myöskin aika pieni jos ruutu on iso.
  • En heti tajunnut että olin siirtynyt yhden noten tarkastelutilaan, ja että takaisin päästäkseni piti painaa "Notes"-tekstiä vasemmassa yläkulmassa. Mielestäni tämä voisi olla jotenkin selkeämmin näytetty.
  • Kaikki toiminnot toimi hyvin.

Koodi

  • Frontendin koodi on selkeää ja hyvin jaoteltua
  • API-kutsun virheenkäsittely näyttää olevan hoidettu kaksin kerroin, kun onError on määritelty mutaatiossa ja silti kutsu on kääritty try-catch-blockiin. En tunne React Queryä kovin hyvin, joten en ole varma, mutta voisi kuvitella että try-catch ei ole tuolloin enää tarpeellista.
  • Backendin koodi on myös selkeää ja hyvin jäsenneltyä. Middlewareja käytetty fiksusti, nosql-injektiokin otettu huomioon. Yksi parannusehdotus on try-catch-koodin poistaminen express-async-errors -kirjastolla, jonka käyttö esiteltiin fullstackopen-kurssilla.
  • Teknologiat on valittu tarkoituksenmukaisesti ja niitä on käytetty hyvin. Tyylit ja responsiivisuus on hoidettu hyvin näppärästi Tailwindilla.

Yhteenveto

Sovellus on selkeä, toimiva ja tarjoaa toiminnot karttanotejen luomiseen. Sovellus on hiottu ja ammattimainen. Koodi on luettavaa, ja jäsenneltyä ja teknologioita on käytetty tarkoituksenmukaisesti. Hyvää työtä!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions