Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sam and Angelica -- Octos #10

Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

Conversation

samanthaberk
Copy link

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C9/VideoStoreConsumer-API#8
For each person in your pair, describe something you learned from your partner during this project Sam: I learned more about passing props to child components, Angelica: I gained more familiarity with functional components
What was one area of React you gained more clarity on during this assignment? Functional components, callback functions, lifecycle methods
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We used ComponentDidMount() to make the API call to get the data, saved the response in the State of our Library Component, then iterated over the state to display the data content to the app
Describe how you handled rentals with React We saved the current/selected customer and movie in the state on App and used callback functions to display and update them. In App we posted to the API to create a rental with the current customer and movie
Describe a DOM event your application handled Toggling the 'create new customer form' display
Did you use any functional stateless components? What for? Yes, we used a functional stateless component for the Status component, because the state can be handled by App so it doesn't need a class
Did you use any container components? What for? App can be considered a container components for Status. It manages the creation of and data sent to Status, a presentation component. Presentation components are only responsible for displaying data, not managing their own state.
Do you have any recommendations on how we could improve this project for the next cohort? It would be interesting if we could have used the APIs we built for the project. A working demo introduced at the beginning would be swell!

amcejamorales and others added 30 commits June 18, 2018 14:54
… VideoStoreConsumer-API on form submit. Mostly functional.
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