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

Leaves - Dianna and Tiffany #8

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

Leaves - Dianna and Tiffany #8

wants to merge 68 commits into from

Conversation

dtingg
Copy link

@dtingg dtingg commented Dec 19, 2019

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C12/video-store-consumer-api#5
For each person in your pair, describe something you learned from your partner during this project Tiffany: Using Postman for GET and POST requests. Dianna: Tiffany is great at figuring out where to keep state and how to make the components interact with each other.
What was one area of React you gained more clarity on during this assignment? Tiffany: Lifting states for states that we don’t intend to store permanently.  We left the states in a lower component. Dianna: I feel more comfortable using nested components in React.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content On the Library and Search pages, we use Movie components.  We passed down different functions using the same callback name to make sure they did what we wanted.
Describe how you handled rentals with React We made POST requests with Axios to generate rentals with our API.
Describe a DOM event your application handled For our Search page, after we submit our form, we update state and the DOM creates the components necessary to render those movies.
Did you use any functional stateless components? What for? Home, Customer, Customers, Library, and Movie are stateless.  We stored the state for these components in App, so they didn’t need to be classes. 
Did you use any container components? What for? Yes, we created Library and Customers to hold collections of Movies and Customers. 
Do you have any recommendations on how we could improve this project for the next cohort? We enjoyed working on this project, but there was some confusion in the beginning about the two different repos.  It would also be helpful to see a finished product.  

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