https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/utilities/utilities.js#L1-L21

When you use someone else's code, make sure to reference it!

README contains minimal documentation. Following headers can be added:

• Actor diagram
• Rate limit
• Interaction diagram
• Design patterns & best practices
• Feature wishlisht & backlog

Looks like your interaction diagram is only for the currentstate of your app.

You'll need to add some functionalities to comply with the rubric requirements.

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L1-L54

JSON word netjes opgehaald met een async function, zover ik kan zien heb je gebruik gemaakt van de iets diepere materie (promiseAll) dit is ook erg netjes.

Je gebruikt nog niet alle: map(), filter(), reduce() methods voor zover ik kan zien. Vergeet niet een functionaliteit toe te voegen waarbij je dit kan gebruiken.

Je hebt je eigen template engine geschreven, dit is zeker een plus.

All in all, denk ik dat je nu nog in de "onvoldoende" zit, maar dit is puur alleen maar omdat je nog niet alle methods gebruikt die we moeten gebruiken. Als je deze gebruikt zit je in de "GOED"

Try to separate the scss and js from the index.js
https://github.com/damian1997/web-app-from-scratch-1920/blob/be25f7714138ba399c89d13cbe1a49ec75180297/src/index.js#L1-L4

https://github.com/damian1997/web-app-from-scratch-1920/blob/be25f7714138ba399c89d13cbe1a49ec75180297/src/js/components/api.js#L35-L36

I would merge these two code lines

API is well documented/explained in your readme, well done. Also the code snippets are a great addition.

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L50-L54

Is this your own code?

And can you explain why this function is available on line 17, while it's declared in line 50?

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L34

It's less verbose and less code

Routie microJS is gebruikt en werkt.

Actor-/Interaction diagrams zijn visueel vorm gemaakt, ik denk echter wel dat deze nog niet 100% compleet zijn.
Vergeet niet wanneer de applicatie af is je diagrams te checken en te kijken of ze nog up to date zijn met je huidige code. Grote kans dat je nieuwe features toevoegt of bestaande feature code refactored.

In de huidige live link valt nog niet veel te zien, vergeet niet een vorm van feedback aan de gebruiker te laten zien om de user experience te verbeteren en te voldoen aan de rubric.

Voor nu zit je nog in "onvoldoende"; dit komt omdat de diagrams uitgebreider kunnen en volgens mij ook nog niet alle features/code bevat.
Als je deze dingen doorvoert zit je waarschijnlijk in "voldoende"/"goed"

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L3

Code is netjes gestructureerd, en je hebt je goed verdiept in nieuwe stof (promiseAll & virtualDom), ook netjes gebruik gemaakt van modules

Ik denk dat je hier makkelijk in de "GOED" zit

In your code, you are using the virtual dom. Make sure to mention and explain this in your readme.

getPage(route) is part of "Template" in your actor diagram, to me this doesn't make sense. Why not make it a part of "Router"?

I suggest visualising variables with a "+" as prefix. This is better readable for users.

• Add a link to your live demo in Github Pages
• Add an image of your work in the readme
• Add a table of contents
• Add a section of how to install the app
• Add the features of the app
• Add an example of the data
• Maybe add a license

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L6

• Classmates is a global variable, so there is no need to use it as a param

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L17

• Well done on creating an async loop, you might want to check .map instead of a classic loop.