Zie issue #5. Repo mist het volgende nog:
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.
Zie issue #5. Repo mist het volgende nog:
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.jsop, leesthandleRoutes()en handleRoutes() splitst op naar twee routes en komt bijoverview()of bijhero/: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.