-
Notifications
You must be signed in to change notification settings - Fork 0
Peer feedback
Twee dagen voor de oplevering was het tijd voor de peer-feedback. Ik kreeg en gaf feedback aan Bas.
Ik heb zijn hele repo gecheckt, inclusief code en de wiki.
Hij heeft hetzelfde voor mij gedaan, daaruit kwamen de volgende punten:
Bas gaf hier aan dat ik meer info kon geven in mijn repo description. Dit heb ik meteen aangepast.
Bas zei dat ik eventueel een extra pagina in m'n wiki kan maken waarin ik het concept van mijn applicatie uitleg. Hier ga ik zeker aan werken als ik tijd over heb.
Hier zei Bas dat ik een screenshot van m'n project in m'n Readme
moet verwerken. Dit heb ik toen meteen gedaan!
In deze issue zei Bas dat ik een linkje neer moest zetten bij mijn bronnen. Ik heb hierna meteen een bronnen pagina aangemaakt en gevuld met mijn gebruikte bronnen.
Bas gaf hier aan dat ik een pagina in m'n wiki moest maken waarin ik meer vertel over wat ik met de data gedaan heb, bijvoorbeeld hoe ik het heb schoongemaakt. Dit ga ik zeker doen als ik er nog tijd voor heb!
Hierin gaf Bas mij een compliment over mijn logboek. Hij gaf wel als tip mee om meer code snippets te laten zien.
In deze issue zei Bas dat ik wat meer research toe moet voegen in mijn wiki. Ik ga inderdaad zeker nog een pagina aanmaken over React en daarin zetten wat ik gebruikt heb.
In de laatste issue zei Bas dat ik meer comments moest toevoegen aan de code. Voor de rest zei hij dat de code er goed uitzag.
Tom van den Berg | 500827055 | Frontend Applications | 2021-2022