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

Ports - Ari and Amy W #25

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

Ports - Ari and Amy W #25

wants to merge 59 commits into from

Conversation

amythetester
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-C11/VideoStoreConsumer-API#18
For each person in your pair, describe something you learned from your partner during this project Ari taught Amy how to use React Router and Amy taught Ari more about callback functions and how to pass information between components.
What was one area of React you gained more clarity on during this assignment? Where to put state in the app and passing functions as props.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We created a Movie component for the rentable movies and a Result component for the search results. This allowed us to pass different functions as props while using the same CSS classes.
Describe how you handled rentals with React We set state in App for the selected customer and the selected movie so that we could use them as params for the checkout method. At this time we had to add the due date and were able to successfully POST. These were then put in a join table in Rails.
Describe a DOM event your application handled One of our onclick methods was selectCustomerCallback which we passed as props to the Customer components. This was able to pass the clicked customer back to the App component and setState.
Did you use any functional stateless components? What for? Yes we did. We used them for Movie, Customer, and Result components.
Did you use any container components? What for? Yes we did. We used them for RentalLibrary, App, Search, and CustomerList.
Do you have any recommendations on how we could improve this project for the next cohort? Modernize the content. Maybe something more like Netflix or Hulu or something not video related. At some point the folks coming to Ada will not be familiar with a Video Rental Store and the processes they used to have to go through.

aribray and others added 30 commits June 24, 2019 14:10
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