Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Full Stack projektin katselmointi #17

Open
Kaltsoon opened this issue Jan 14, 2022 · 1 comment
Open

Full Stack projektin katselmointi #17

Kaltsoon opened this issue Jan 14, 2022 · 1 comment

Comments

@Kaltsoon
Copy link

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
Copy link
Owner

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants