Zorg dat je code modulair wordt, alles is nu gewoon op root niveau gescoped en wordt synchroon van boven naar onder uitgevoerd
Data ophalen, manipuleren en tonen
Voldoende
De functionaliteit die er in zit is wel compleet gefocust op de API en zie dat je hier waarschijnlijk veel moeite in hebt gestopt. Je manipuleert je data zodat het in je eigen structuur staat wat heel handig kan zijn, al ziet het er voor nu een beetje langdradig uit. Probeer nog gebruik te maken van een templating engine of een specifieke render functie in plaats van die lange html string alleen adjacentHtml toe te voegen.
States en routes afhandelen
Onvoldoende
Routing werkt in principe, je kan het nog een beetje uitbouwen naar meerdere verschillende routes en states toevoegen. Dit lijkt me een prima haalbare to-do als je verder aan je app werkt en de structuur ook fixt
App structuur en code kwaliteit
Onvoldoende
De structuur van je code lijkt nog vrij quick-and-dirty, je ziet duidelijk dat je nog gefocust bent op functionaliteit werkend krijgen en nog weinig aan structuur hebt gedaan. https://github.com/deannabosschert/web-app-from-scratch-1920/blob/2440c2af26dd613ec443be412ecfd4b8493c3a3a/docs/js/app.js#L33-L341 Dingen als de HTML toevoegen met een grote 'som' van strings is zeer onleesbaar en lijkt mij ook heel vervelend om mee te werken.
Zorg dat je code modulair wordt, alles is nu gewoon op root niveau gescoped en wordt synchroon van boven naar onder uitgevoerd
Data ophalen, manipuleren en tonen
Voldoende
De functionaliteit die er in zit is wel compleet gefocust op de API en zie dat je hier waarschijnlijk veel moeite in hebt gestopt. Je manipuleert je data zodat het in je eigen structuur staat wat heel handig kan zijn, al ziet het er voor nu een beetje langdradig uit. Probeer nog gebruik te maken van een templating engine of een specifieke render functie in plaats van die lange html string alleen adjacentHtml toe te voegen.
States en routes afhandelen
Onvoldoende
Routing werkt in principe, je kan het nog een beetje uitbouwen naar meerdere verschillende routes en states toevoegen. Dit lijkt me een prima haalbare to-do als je verder aan je app werkt en de structuur ook fixt