feedback
- Klasgenoot Marc Bakker gaf mij de tip om te kijken naar het schrijven van code die er voor zorgt dat als ik iets geselecteerd heb in mijn homepagina, de regel groen wordt om er voor te zorgen dat de gebruiker weet dat het dit heeft aangepast.
- Klasgenoot Marc Bakker gaf mij de tip om mijn zoekresultaten aan te passen zodat de regels niet lijken op knoppen. Ik heb dit gedaan door de borderradius naar 0 te zetten en de achtergrond kleur van blauw naar grijs aan te passen.
Peer review for a1 Reviewer Marc Bakker Reviewee Koen Dekker Class Tech 3 Teacher Robert Spier
- Source code is available on GitHub Ja
- Project is documented and has a readme.md Ja
- Cites the sources used; code comments or docs Nee
- Research is documented in the wiki Ja
- A live version is deployed and linked in description N.v.t.
Git clone the repository of your fellow student and install it following the installation guidelines from the readme. Can you get it up and running? Does the readme.md miss essential information?
"De meest recente code staat nog niet online. En hij wilt nog zijn API key apart opslaan voor privacy redenen, dus werkt de feature nog niet via het downloaden van zijn GitHub Repo. Dus nee."
"J. & R. Midden, alles staat erop wat er voor gevraagt wordt." "Inter, HTML breakdown geeft pluspunten. Alles gedaan wat er verwacht wordt."
- Is there any documentation on the git strategy? Ja
- Is there any documentation on the local dev environment? Ja
- Is there any documentation on deploying and production? N.v.t./Ja
"Alles gedaan wat er verwacht wordt. Veel uitleggende tekst zodat het punt duidelijk wordt."
"Alle knoppen werken. Dus je kan inloggen, zoeken. Je krijgt de resultaten, deze zien eruit als knoppen. Maar omdat het maar een feature is werkt dat niet."
- Does the readme.md cover what the project does? (concept) Ja
- Are there any linters installed and used in the project? Ja
- Is version control user properly? Commits early and often. Ja
Comments "Code: Comments maken de code duidelijk. Linters, Formatters helpen ook. Deze worden goed gebruikt." "Documentatie: Documentatie ziet professioneel uit. En is volledig ingevuld."