Peer review - Sanne 12 feb

[sanneduinkerx]

Repo

• Leuk concept!
• Repo bevat de link naar live demo :)
• Bevat nog geen description
• Repo bevat uitgebreide uitleg over de API van spotify en is duidelijk

Actor Diagram

• Actors zijn duidelijk, ik snap de functions per actor en is logisch

Interaction Diagram

• ik mis alleen in de interaction diagram de render.js actor 😁
• en misschien nog toevoegen wanneer er een fout is bij data laden dat je dan iets anders laat zien.
• Verder is het voor mij wel te volgen!

Code

• Tip: dat noemde je net ook, dat je wat comments zet bij bepaalde functies en modules zodat het ook makkelijker is om uit te leggen later wat er gebeurd. 😄 (en voor mij ook beter leesbaar XD)
• missen nog actors van wat er in de diagrammen staan.
• In de main.js kan de code nog opgesplitst, maar volgens mij is dat ook je plan. 😊
• Benamingen van functions en variable zijn voor mij duidelijk!
• Ik zelf zou renderen van de html anders doen met createElement en appendChild, maar dat is per persoon anders denk ik. 😊