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 - Dominique & Yasmin #13

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

Conversation

dtaylor73
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-C12/video-store-consumer-api#6
For each person in your pair, describe something you learned from your partner during this project Dominique learned the format of react routes from Yasmin. Yasmin learned how to debug from Dominique.
What was one area of React you gained more clarity on during this assignment? We gained more clarity on how React coordinates interaction between all of its components and how React interacts with the rails API.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We rendered the overview the movie to determine the differences between each movie.
Describe how you handled rentals with React We made a get request to the Rails API inside of a componentDidMount method.
Describe a DOM event your application handled OnClick events and OnSubmit
Did you use any functional stateless components? What for? The customer component is a functional component and stateless. We made it functional because its only functionality is to render html.
Did you use any container components? What for? Yes. Customer List passes down props to the customer component. Movie Search passes down props to the movie results component. Rental Library passes down props to the movie component.
Do you have any recommendations on how we could improve this project for the next cohort? No recommendations.

YasminM11 and others added 30 commits December 16, 2019 19:18
Customer branch - Latest Pull Request
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