peer review

[ofahchouch-gh]

Er is niet heel veel tijd genomen voor de readme zie ik, maar zoals je weet mist het nog wat. De mijne mist ook nog wat. Het volgende mist:

• geen beschrijving van de socket events
• geen link naar productie omgeving
• geen .gitignore (voor node modules, etc.)
• geen data-flow diagram

Je hebt een leuk concept bedacht begreep ik uit de meetings. Maar in de repo is het niet heel duidelijk. Je legt uit hoe de app werkt middels de tekeningen. Je had als CMD student wel wat meer tijd er in kunnen steken :P. Als je een API gaat gebruiken om woorden op te halen zou ik die ook in je readme omschrijven.

Verder nog een tip over de code. Probeer met modules te werken, dat gebeurd straks alleen maar op stage/werk. Dan is het een goede gewoonte.

[HappyPantss]

Hey Oussama!

Dankjewel voor de feedback! De readme dingetjes heb ik al verwerkt, of heb ik een opstelling voor op gezet zodat ik dit niet ga vergeten en daar word zo snel mogelijk aan gewerkt!

En klopt, ik had wat creativer mijn idee kunnen beschrijven, dat doe ik namelijk al bijna 3 jaar, beetje slordig!

En klopt, ik zou met modules moeten werken, goede tip dankjewel!