Alrefai / jammming

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Grade: Exceeds Expectations

Rodrigap opened this issue · comments

Really nice job on this project! Your app works great and your codebase is one of the more creative ones I've seen for this project. I love that you added extra features, like the ability to search on enter and the removal of the playlist tracks from the search results. The second feature has a couple of minor bugs, but it's still great that you included it. My biggest critique is that I didn't love how you separated components versus containers, but that could just be a personal preference. Overall, this was really creative and well done! Great job

Thanks for your delightful review. I am honored!

Regarding components versus containers, I thought that was a good practice. I have learned that in one of the advanced React courses in Codecademy. However, I understand your point of view. This is a simple app and I should have kept it simple. I was just excited to put what I have learned into practice.

P.S. that bug is fixed. Please refer to issue #4.