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

Jazz & Maryam #21

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

Jazz & Maryam #21

wants to merge 22 commits into from

Conversation

jazziesf
Copy link

@jazziesf jazziesf commented Dec 24, 2018

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API http://localhost:3000
For each person in your pair, describe something you learned from your partner during this project
M - I felt like I got a good example of project ownership and great workflow working with Jazz, I also learned about embedding audio. J - I think we were a good compliment of each other in what we wanted to individually work on as well as her patience to work on the majority of the project together. We were stuck on callbacks for sometime and at no time did she seem flustered we just keep trying to make sense of it together.
What was one area of React you gained more clarity on during this assignment? M- The React Router, and a better understanding of how to design Component relationships and passing data around. J - The use of breaking down components to not flood your application. Callbacks through the app from two different components. How an backend works with a separate frontend.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We used a stateless movie component with props to pass the information we needed for the different functionality especially with the callback.
Describe how you handled rentals with React We created a Checkout component that is rendered by App and gets movie and customer objects as props. Checkout has an onRental function that is called when the user clicks the checkout button, it sends a post request to the rails API with the required params (customer id, movie title and due date).
Describe a DOM event your application handled When you click a customer to select it, a callback function is initiated in the AllCustomers component that sends the clicked customer to the App component. App saves the customer in it’s state and eventually passes the customer to Checkout.
Did you use any functional stateless components? What for? Movies and Customer are stateless components we used to define individual instances of movie and customer objects. Navbar and Search are also stateless components.
Did you use any container components? What for? Yes, we used RentalLibrary as a container component for requesting and managing all available movies and AllCustomers for managing all customers.
Do you have any recommendations on how we could improve this project for the next cohort?

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