peer review dubbele beoordeling #1
Description
APPLICATION FD
- de view op de data verander je nu door te zoomen, misschien is het interessant om te kijken of je een gebied kan selecteren en daar dan de data van weergeeft in plaats van dat je op dat gebied inzoomt
- het is nog niet duidelijk wat de rode bolletjes betekenen of hoe de visualisatie werkt.
- je hebt de voorbeeldcode zo doenlijk aangepast dat het je eigen code is geworden, super tof!
APPLICATION FP
- je gebruikt regelmatig map en filter en de functies zijn goed herbruikbaar door gebruik van parameters en globale benamingen
- data wordt asycn ingeladen met promisses
- Je comments leggen duidelijk uit welke stappen jij neemt om de data op te schonen
UNDERSTANDING FP
- uitleg van FP is aanwezig inclusief voorbeeld, je legt goed in eigen woorden uit wat FP is. Je zou eventueel nog een paar voorbeelden kunnen toevoegen zoals bijvoorbeeld hoe je dit in je d3 code toepast
PROCESS
- je zou nog wat meer uitleg en struggles kunnen documenteren bij je curran course pagina. Ook zou je nog wat informatie over map en filter kunnen schrijven, nu gebruik je alleen voorbeelden
- ik denk dat je visualisatie zeker duidelijk kan zijn voor je doelgroep als je nog wat meer informatie toevoegt zoals een legenda en misschien hoe je de visualisatie gebruikt.
QUALITY
- Je readme is duidelijk, je zou nog instructies kunnen toevoegen over hoe je het project installeert (git clone enzo)
- de structuur van je wiki is duidelijk en de pagina's lezen fijn. kijk wel even naar je wiki home pagina, deze is in het engels terwijl de rest van je wiki in het nederlands is
- je naming conventions zijn goed en verklaarbaar, alleen soms zijn ze in het engels en soms in het nederlands. probeer hier consistent in te zijn
- verder heeft je code een duidelijke opbouw, goede comments (misschien nog even alle uitgecommente console logs weghalen) en is de code goed te begrijpen.
overall, super cool project ✌🏻