search

nmaurok / jamming-codecademy

codecademy intensive project
0 stars 0 forks source link

Summary #4

Open thecodeduchess opened 6 years ago

thecodeduchess commented 6 years ago

Grade: Needs some improvement

Summary:

I was not able to run your project's search feature (or add anything if I can't get search/spotify to work). Please check spotify and your search functions again. You may want to leave tokens to redirect to localhost:3000 since that will always be available for my computer locally.

Everything else seems to be in order. It seems like you've got a good handle on React syntax. Your consistency with indentations and whitespace for structured code made it legible and easy to navigate through.

You are allowed to change the background, color scheme or fonts to your liking so don't be afraid to play around with that. You can make it your own.

This was a great attempt so I hope you are enjoying your learning experience. Good luck with your feature request!

nmaurok commented 6 years ago

Thank you for your feedback! Btw, is it necessary for me to fix the app now in order to get the certificate? I'm busy with exams and I'm not sure if I'll be able to manage.

thecodeduchess commented 6 years ago

No. You did it so that should be fine.

On Saturday, November 11, 2017, nmaurok notifications@github.com wrote:

Thank you for your feedback! Btw, is it necessary for me to fix the app now in order to get the certificate? I'm busy with exams and I'm not sure if I'll be able to manage.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nmaurok/jamming-codecademy/issues/4#issuecomment-343712125, or mute the thread https://github.com/notifications/unsubscribe-auth/AIUaFk00KPzopkVFcMkzrc5AvRS4LHv6ks5s1m-1gaJpZM4QavFi .

-- Tell me and I forget. Show me and I remember. Involve me and I understand.