[REPO] Update repo

[bommezijn]

Zie issue #5. Repo mist het volgende nog:

• Poster image
• Table of Content
• Kopje waar je de demo kan vinden in de readme.md
  • Devs zijn lui dus die vinden het wel prettig dat alles bij elkaar staat
• Peer review web-app-from-scratch-2021 week 2 #5 beschrijving wat voor applicatie het is
• Wat is het concept?
• Hoe installeer ik de applicatie?
  • Wat heeft een gebruiker nodig voordat ik deze applicatie kan gebruiken
• Hoe gebruik je de applicatie?
  • Wat zijn de features?
  • Welke features wil je erin en welke laat je eruit voor de toekomst?

Onder welke license valt dit project? Op dit moment kan ik de repo clonen en zeggen dat ik het heb gemaakt.

Diagrammen

Diagrammen reflecteren jouw codebase niet. Het doel van de actor diagram is dat je in een oogopslag kan zien hoe je code opgebouwd is maar op dit moment wil ik van app.js naar renderData.js maar deze bestaat niet.
Refactor de benaming van je namen en functies of wijzig de diagram.

Mijn tip hiervoor is, kijk naar wat je code doet, html roept app.js op, leest handleRoutes() en handleRoutes() splitst op naar twee routes en komt bij overview() of bij hero/:name.

Top dat je alvast toekomstige functies erin plaatst.

Als laatst voor de diagrammen; Op dit moment leest het alsof je objecten of classes gebruikt maar je gebruikt functies, dus de dot notation voor je functies moet weg tenzij het daadwerkelijk een class is of een method in een functie.

[bommezijn]

Excuus voor de lange issue.

Also bedankt, zie in je router.js op L6 dat ik de default route heel anders kan aanpakken.