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 - Ga-Young, Bri #26

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

Leaves - Ga-Young, Bri #26

wants to merge 42 commits into from

Conversation

brilatimer
Copy link

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API We did not get to the feature of saving a movie to the Library via the endpoint API.
For each person in your pair, describe something you learned from your partner during this project Bri learned how to break down bugs and be more persistent in her approach (like Ga-young!). Ga-young learned more about the React overall connection process and how it flows by talking through it with Bri.
What was one area of React you gained more clarity on during this assignment? Both: learned more about how callbacks function, raising state to a higher component
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Making Movies a component (so title is included, but different for each movie)
Describe how you handled rentals with React Rentals is a component. To create a new rental, you send a POST request to the API
Describe a DOM event your application handled ONCLICK, ONSUBMIT, ONCHANGE
Did you use any functional stateless components? What for? Customer, Movie, Rental, do not have state, just the information on what to render.
Did you use any container components? What for? Customer Collection, Movie Collection, Rental Collection. These were used to tell React how to present the data (table of customer information etc).
Do you have any recommendations on how we could improve this project for the next cohort? Clarification and further details on what parts of the API need to be changed.

gyjin and others added 30 commits December 16, 2019 15:02
added ability to see all movies and select one
can see overdue rentals
selected movie from Search will appear in library
error messages for making a new rental
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