search

mikeleppane / go-explorer

New, intuitive, and fun online environment for learning, exploring, and experimenting with the Go programming language.
https://golangexplorer.org
MIT License
18 stars 0 forks source link

Full Stack projektin katselmointi #17

Open Kaltsoon opened 2 years ago

Kaltsoon commented 2 years ago

Full Stack projektin katselmointi

Terve @mikeleppane! 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?

Kokeilin käyttöohjeen mukaisi toimintoja.

Kokemus

Sovelluksen käyttöliittymä on erittäin tyylikäs ja intuitiivinen. Toiminallisuus toimi moitettoomasti. Keyboard shortcutit voisi olla hyödyllisiä eniten käytetyissä toiminnoissa, kuten koodin suorittaminen. En ainakaan pikaisella käyttöohjeen selailulla huomannut, että ne olisivat käytössä. Automaattinen linttaus jollain järkevällä debounce-ajalla voisi myös olla kätevä. Geneerisen "React App"-faviconin voisi muuttaa kuvaavammaksi.

Koodi

Koodi on kaikin puolin selkeästi organisoitua ja teknologioita, kuten TypeScript ja kirjastoja, kuten Material UI on hyödynnetty järkevästi. Koodin laatu on korkea, esim. komponenttien koodi on pidetty selkeänä ja kompaktina hyvien abstraktioiden avulla. Testaus on kattavan näköistä sekä yksikkö-, että integraatiotestien osalta.

mikeleppane commented 2 years ago

Kiitokset kommenteista! Ehdottamasti keyboard shortcutit on selkeä puute. Tämä tullaan korjaamaan ensi tilassa. Hyvä ehdotus tuo automaattinen linttaus!