Week 7 - Code review
Name
Sergio Eijben
Class
Tech-1 (204)
Feedback
Team: Het is goed dat jullie een repo hebben geforked. Iedereen heeft een eigen branche waar hij/zij in kan werken.
Individueel: Het werk wat je hebt gemaakt ziet er erg netjes uit. Je gebruikt veel comments om dingen uit te leggen. Alleen gebruik wel 1 taal.. Je README staat bijvoorbeeld in het Engels maar ook in het Nederlands? En je comments staan in het Nederlands. Maar dat kan kloppen omdat veel namen van je classes en functies ook Nederlands zijn. Maar let hier dus wel op!
Ook heb je in je .gitignore wel staan dat je .env en node_modules wilt ignoren, maar deze staan nog op je repo!
De command die bij "3. Run!" staat klopt ook niet (nodemon). Ik kon dan toevallig in je package.json zien wat het wel was (node index.js).. maar update dit dus!
Feedback specifiek op criterium 1 Project Tech
Individueel: Je hebt goed gebruik gemaakt van de branches. Je hebt die voor jezelf en Ralf gemaakt. Hierdoor kan iedereen zijn eigen ding doen. Ook doe je vaak zat commits, en dat is goed!
Feedback specifiek op criterium 2 Project Tech
Ik zie in jullie team repo dat jullie netjes alle calls bij houden die jullie gedaan hebben. Hier staat genoteerd waar jullie het over gehad hebben. De code standaarden die jij gebruikt zien er goed uit. Je code is netjes ingedeeld en goed uitgelijnd!
Feedback specifiek op criterium 3 Project Tech
Je documentatie is tot zo ver duidelijk, er missen alleen wat dingen zoals een afbeelding en wat uitleg over de database connectie en uitleg ove het includen van een .env en .gitignore file.
Feedback specifiek op criterium 5 Project Tech
Het is logisch dat jullie nog niks gemerged hebben, dus daar geef ik ook geen feedback over. Je persoonlijke feature ziet er wel super goed uit! Ook werkt alles goed.
Feedback over andere dingen, zoals voor backend of frontend
De frontend van je app ziet er erg goed uit, alles is erg duidelijk. De backend is ook super nice. Als je inlogt en vult iets verkeerd in, kan je dit zien in de console! Je kan (als het goed is? Geen idee hoe ik dit check) je account veranderen, verwijderen en je zelf uitloggen.
Week 7 - Code review
Name
Sergio Eijben
Class
Tech-1 (204)
Feedback
Team: Het is goed dat jullie een repo hebben geforked. Iedereen heeft een eigen branche waar hij/zij in kan werken.
Individueel: Het werk wat je hebt gemaakt ziet er erg netjes uit. Je gebruikt veel comments om dingen uit te leggen. Alleen gebruik wel 1 taal.. Je README staat bijvoorbeeld in het Engels maar ook in het Nederlands? En je comments staan in het Nederlands. Maar dat kan kloppen omdat veel namen van je classes en functies ook Nederlands zijn. Maar let hier dus wel op!
Ook heb je in je .gitignore wel staan dat je .env en node_modules wilt ignoren, maar deze staan nog op je repo!
De command die bij "3. Run!" staat klopt ook niet (nodemon). Ik kon dan toevallig in je package.json zien wat het wel was (node index.js).. maar update dit dus!
Feedback specifiek op criterium 1 Project Tech
Individueel: Je hebt goed gebruik gemaakt van de branches. Je hebt die voor jezelf en Ralf gemaakt. Hierdoor kan iedereen zijn eigen ding doen. Ook doe je vaak zat commits, en dat is goed!
Feedback specifiek op criterium 2 Project Tech
Ik zie in jullie team repo dat jullie netjes alle calls bij houden die jullie gedaan hebben. Hier staat genoteerd waar jullie het over gehad hebben. De code standaarden die jij gebruikt zien er goed uit. Je code is netjes ingedeeld en goed uitgelijnd!
Feedback specifiek op criterium 3 Project Tech
Je documentatie is tot zo ver duidelijk, er missen alleen wat dingen zoals een afbeelding en wat uitleg over de database connectie en uitleg ove het includen van een .env en .gitignore file.
Feedback specifiek op criterium 5 Project Tech
Het is logisch dat jullie nog niks gemerged hebben, dus daar geef ik ook geen feedback over. Je persoonlijke feature ziet er wel super goed uit! Ook werkt alles goed.
Feedback over andere dingen, zoals voor backend of frontend
De frontend van je app ziet er erg goed uit, alles is erg duidelijk. De backend is ook super nice. Als je inlogt en vult iets verkeerd in, kan je dit zien in de console! Je kan (als het goed is? Geen idee hoe ik dit check) je account veranderen, verwijderen en je zelf uitloggen.