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 - Cloudy and Morgan #29

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

Conversation

morganschuler
Copy link

@morganschuler morganschuler commented Dec 23, 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#21
For each person in your pair, describe something you learned from your partner during this project Morgan: Cloudy taught me about react strap and how you can use both bootstrap and react together! Cloudy: I learned about react strap and how to implement the search.
What was one area of React you gained more clarity on during this assignment? We gained more clarity on react routers! That was something we hadn't seen before.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content They worked as separate entities, so we stored our own list of rentals in the application but then imported from the DB.
Describe how you handled rentals with React We created an event that upon clicking the movie would add rentals to the library.
Describe a DOM event your application handled We created OnClick events for our application.
Did you use any functional stateless components? What for? Yes, Movie and customer because they did not hold state.
Did you use any container components? What for? CustomerList and Library. They held state and would render the props.
Do you have any recommendations on how we could improve this project for the next cohort? none that we can think of!

morganschuler and others added 28 commits December 16, 2019 15:53
…tomer to create table. its all up and running
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