Skip to content

Conversation

@njch5
Copy link

@njch5 njch5 commented Dec 20, 2019

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API https://github.com/njch5/video-store-consumer-api
For each person in your pair, describe something you learned from your partner during this project Samantha: Nicky is great at organizing the JavaScript functions and very knowledgeable about JavaScript/React syntax. Nicky: I learned a lot from Sam on CSS and I thought she did a fantastic job with the front-end piece of the project. We pair programmed on defining the relationships between the components in order to see where the callback functions and props are.
What was one area of React you gained more clarity on during this assignment? We gained clarity on the callback functions and how to pass the props between the child and parent components.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We created a MovieCollection component which maps over the movies we retrieve from the GET request from axios and then in the App.js file we render the list of movies.
Describe how you handled rentals with React We created a RentalDetails component which will display the selected customer and movie, one by one, then display a button to create a rental. In the App.js file, we have a addRental function which will make a POST request to axios to create a new rental.
Describe a DOM event your application handled When selecting a customer, we have an onClick prop that will send a callback to the App.js (parent component) and will trigger the selectCustomer function to set the current customer to that particular event.
Did you use any functional stateless components? What for? We had a MovieCollection component which did not hold state but rather mapped over the movie objects in order to display the pertinent information about each movie.
Did you use any container components? What for? We did not use any container components.
Do you have any recommendations on how we could improve this project for the next cohort? Provide a wireframe that shows how the user experience flow should go.

njch5 and others added 30 commits December 16, 2019 15:03
Edited fields for movie to show image and external ID
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