Thanks!

[wawgoingdad]

Oops the GIF is broken :)

[frost4869]

Yah sorry about that. Im gonna commit another one to github soon !

On Tue, Nov 28, 2017 at 3:19 PM, Huy Mai notifications@github.com wrote:

Oops the GIF is broken :)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/frost4869/flixie-moviedb/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AP2u4X7TfmZ7EfF3Wb8Q5_wYTziRoxpJks5s68IXgaJpZM4Qs2HB .

-- Nguyen Vu Hoang Son, FPT University, Web Developer.

[coderschoolreview]

Hello! Thank you for submitting the first homework for CoderSchool React Class 🎉! The main focus of this assignment was to introduce you to working with an API, rendering items in a collection, and dealing with state and props.

Thanks for creating a nice README.md with the checkboxes. It makes it a lot easier to understand the functionality of your app.

• Good job fetching the data in App.js. Usually it's recommended to do this is in componentDidMount. The official Facebook React Native example does this as well. (React and React Native have the same convention here.)
• It's great when you're using template string. Awesome way to format string.
• Cool job using Semantic UI library. It's nice to use a CSS Framework because you can build off base styles easily and quickly.
• Remember to not keep any component too long! You did a good job splitting up into MovieList, MovieCard,... There may be another opportunity to split MovieSearch or something like that
• It's totally fine to have inline style like "style={{ fontSize: 53, fontWeight: 'bold' }}" but defining a constant with different styling properties would be preferred for a easier reading and maintaining code.