-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Exceeds expectations
I am blown away, I assume you knew some react before taking this course, this is hands down one of, if not the best jamming project's I have reviewed, and I have reviewed a lot of them. The added styling was icing on the cake! I love that you added comments to guide me through the code, and it looks like you even refactored getaccesstoken as well. Some project I will leave issues for alternative code, yours I didn't find much, one thing I will note is in addTrack in the app.js file, you could have used .find() for selecting the track, no big deal there. Overall this was well done and I hope you do a good feature to add on!
Metadata
Metadata
Assignees
Labels
No labels