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

Sockets - Chantal and Niv #15

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

Conversation

nidhiparixitpatel
Copy link

@nidhiparixitpatel nidhiparixitpatel commented Jun 28, 2019

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#10
For each person in your pair, describe something you learned from your partner during this project Niv: I learned about how to do the search bar and filter results through a form and how to style a navigation bar with bootstrap from Chantal. I learned more about state and how to better utilize the App.js file for future projects.
What was one area of React you gained more clarity on during this assignment? We learned about how to create a React Router to connect different pages on the front end.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We made a singular movie component that was generic enough to display in both the search list and the library list. The difference between them, which was mostly the button, we put in the parent components Search and Library so that they do not conflict with each other.
Describe how you handled rentals with React We had a state that captured information for the selected movie and customer and we created a function checkout that sent a post request to the api to check out the movie.
Describe a DOM event your application handled We had a button for select customer that takes information from the Customers component and sends it to App which then changes the state and displays it in the navigation bar under selected customer.
Did you use any functional stateless components? What for? No we did not, I think we could have maybe made Movie stateless.
Did you use any container components? What for? Yes, Search, Library, and Customers are all container components that hold components Movie or Customer.
Do you have any recommendations on how we could improve this project for the next cohort? Clearer directions.

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