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- Maria + Kate #14

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

Conversation

KateAnnNichols
Copy link

@KateAnnNichols KateAnnNichols 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#9
For each person in your pair, describe something you learned from your partner during this project Kate - I gained a better understanding of routing. Maria - I gained a better understanding of how to make sites more dynamic.
What was one area of React you gained more clarity on during this assignment? Handling events with React elements
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We used bootstrap, and each individual movie is passed through to an individual card to display with its information and an image of the movie cover.
Describe how you handled rentals with React We made an axios post, once we had a customer and a movie selected, then the rental button appears on the page, can be clicked, and populates the URL parameters to do a post request.
Describe a DOM event your application handled Having the selected name and selected movie, then the rental option
Did you use any functional stateless components? What for? movies and customers
Did you use any container components? What for? Yes, we used container components to hold the response - movie list, movie search, and customer list.
Do you have any recommendations on how we could improve this project for the next cohort? It would be useful to have a quick introduction to react routing before the project.

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