sjorswijsman / web-app-from-scratch-2021

Web App From Scratch @cmda-minor-web 2020 - 2021

Hi there 👋

I am a Web Dev & Designer based in Utrecht, The Netherlands. My passion has always been creating creative solutions to help other people. Recently finished studying Communication & Multimedia Design at Amsterdam University of Applied Sciences. Check out my CV here!

web-app-from-scratch-2021's Issues

Error handling

web-app-from-scratch-2021/scripts/authUser.js

Line 75 in fcf8f5b

if (!data.error) {

Werkt zo ook wel, maar is niet echt een nette manier om op deze manier error handling te doen. Hier heb je in JavaScript weer andere manieren voor. Zie bijv. https://javascript.info/promise-error-handling & https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/try...catch

Daarnaast is een console.log natuurlijk niet echt een handige manier om feedback te geven aan de gebruiker. Misschien kun je hier nog iets anders mee doen.

Code review week 1

Yo Sjors,

Spotify! Relatief uitdagende API. Je ben het verst van alle Spotify uitwerkingen die ik voorbij heb zien komen! Heel nice.

Goeie overzicht pagina! Ziet er clean uit.

Je authUser.js is ook netjes. Je bent een van de weinige die het Spotify authentication gedeelte op een verzorgde manier in zijn code heeft zitten.

Code netjes gestructureerd en gecomment!

Modules helpen het overzichtelijk te maken, ga zo door.

Paar dingetjes:

• Ruim je repo wat op. Er zitten nog een aantal dingen in je repo van de originele fork. Gooi alle dingen die je niet gebruikt lekker weg, schept wat ruimte en overzicht.

• Breid je readme uit. Je hebt je livelink en een korte omschrijving van het concept beschreven. Het zou nice zijn als je wat vertelt over de data die je ophaalt en gebruikt.

• Je zou je website op een andere plek dan authUser.js kunnen 'kickstarten'. Dit is meer een dingetje voor het overzicht en de duidelijkheid, het is natuurlijk niet fout hoe je het nu doet. Je zou hier even naar kunnen kijken. Waterfall vs. Returning values.

Voor de rest heb ik weinig te melden over je code. Gaat lekker!

Comments

web-app-from-scratch-2021/scripts/authUser.js

Line 34 in fcf8f5b

// https://stackoverflow.com/questions/14867835/get-substring-between-two-characters-using-javascript

Zie wel vaker van dit soort comments staan. Zijn dit nog TODO's?

Enter bij css custom props zetten

web-app-from-scratch-2021/style/main.css

Line 12 in fcf8f5b

font-size: calc(60% + 0.8vmin);

Het maakt het in mijn optiek leesbaarder door hier even onderscheidt te maken in custom properties en andere waardes. Extra enter moet voldoende zijn.

Peer review - 05/02

Repository

• Scan your buddy's repository

• Is there a repo description at the top? --> missing, describe the assigment and your concept in the Readme, so the user knows what your project is and how it's organised

• Is the repo open to issues?

• Does it include a gitignore

• Is the concept described well in the readme? --> missing, also mentioned.

• Is the API described in the readme? --> missing, a documentation about what data you use is crucial ;)

Tip:

• Put the link of your live app (deployed on GH pages I see) in the repo description at the right side.

Code

• Read the code
• Try to understand the code flow
• Check the code against our course's best practices --> You're doing well!
• Check the code for consistency, is code being written the same way in different places?
• Does the code follow naming conventions? Do variable and function names make sense to you?
  Awesome, when i login in at your app, I see my own playlists, sick 🤓 . You have clean and clear code. It has very descriptive names for the functions and variables and you work with ES6 modules, nice! I like your code flow!

Tip:

• You are writing code comments for functions, very nice! But write is also for the variables, that makes it more clear :)

Good job buddy! 🚀 🚀 🚀

Modules

Als ik naar je code kijk denk ik bij mezelf zit Sjors nog op deze minor of is die daar al klaar mee? Ziet er echt top uit en ik kan er zelf heel veel van leren. Moet binnen mijn eigen project nog modules toepassen en dat kan ik zeker doen met die van jouw als voorbeeld!

Code review

Mooie app, ziet er erg goed uit! Een paar tips:

• Wat is precies je concept? Hoe kun je straks nummers gaan ranken? Ik ben benieuwd, werk dat voor jezelf nog even uit (door bijvoorbeeld een schets te maken) en neem dit mee in je READme
• Nice van de bronvermelding in de code comments
• Jonah heeft ervoor gezorgd dat de API token niet meer in de URL komt te staan, leuk om eens te checken (nice to have)
• Complimenten hoe overzichtelijk en DRY de code is!