-
Notifications
You must be signed in to change notification settings - Fork 0
.feedback
Micheal:
- Wiki is nog niet af
- Weinig onderzoek naar API's
- Maak duidelijk welke methodes je hebt gekozen, waarom je ze hebt gekozen en hoe je implementeert
Robert
- Documentatie is goed geschreven
- Comments missen bij design
- Comments missen bij interface
Micheal:
- API onvoldoende geïmplementeerd
- Geen Combinatie van API's
- Weinig DOM-manipulatie
- Geen fallbacks
- JS gebruikt nog ES5 code
- HTML niet semantisch
Ivo:
"Het is fijn te zien dat je stappen vooruit hebt weten te zetten. Op dit moment is het resultaat echter nog niet voldoende. Er is maar een beperkte hoeveelheid backend code en de code die er is. is zo opgezet dat elke functie alleen in staat is, om om te gaan met 1 heel specifiek stukje content. Dit is geen werkbare aanpak om een site te bouwen, want je kunt onmogelijk functies blijven schrijven voor elk nieuw concert in de agenda. Werk in plaats hiervan met een algemener stuk code, waarbij een variabele wordt gebruikt om te bepalen om welk concert het dan precies gaat. Van de code die er is, kun je op dit moment ook nog onvoldoende de werking uitleggen.".
Wiki for Blok Tech | Matthew Tangkilissan | 500820664 | klas 203 |