Skip to content

Conversation

@arrriiii
Copy link

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API https://github.com/Ada-C9/VideoStoreConsumer-API/pulls
For each person in your pair, describe something you learned from your partner during this project Ari- Luxi showed me how to pass in attributes from an object to onClick-handler. Luxi- I learned how to create a boolean button for the movie search page.
What was one area of React you gained more clarity on during this assignment? We learned how React and React Router interacts with a back-end rails api & database.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content By building another component that rendered the content to be displayed.
Describe how you handled rentals with React We used a rental container that was called in App.js. The rental container would handle the state of the rental, post request to submit a rental and render each component.
Describe a DOM event your application handled Our movie component has an onClick-handler that would display the correct button for the movie library or movie search page.
Did you use any functional stateless components? What for? We have a functional stateless component for status messages as it only displays the message and type.
Did you use any container components? What for? We did not used a container component in this project-- container components only manage data and render corresponding sub-components.
Do you have any recommendations on how we could improve this project for the next cohort? Do this project earlier in the curriculum.

arrriiii and others added 30 commits June 18, 2018 12:02
…vie.js component to render the movie title and poster image.
… Search.js and added axios to get the url with input.
…ginning so we added a split on the image url to render the correct http link to show the photo on the library page
RentalContainer, customercallback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants