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

Richelle and Rachael - Water #23

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

Conversation

r-spiel
Copy link

@r-spiel r-spiel commented Jan 22, 2021

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C14/video-store-consumer-api#18
For each person in your pair, describe something you learned from your partner during this project Rachael ~ the difference between Radio and Select buttons. Richelle ~ Bootstrap trickiness.
What was one area of React you gained more clarity on during this assignment? Javascript/React events.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content The library component renders individual video components by mapping through the array of objects.
Describe how you handled rentals with React App holds state for a selected customer and video and then does post request to rentals.
Describe a DOM event your application handled The search componenet.
Did you use any functional stateless components? What for? ?
Did you use any container components? What for? ?
Do you have any recommendations on how we could improve this project for the next cohort? This was a difficult project and 2-3 days was not enough time. We felt a lot of the backend was very challenging and did not have a lot of time to work on the frontend, but we are presenting it.

r-spiel and others added 30 commits January 19, 2021 15:33
render list of customers from local api
Everything showing for video and library, the beginning of the search…
adding state for selected customer to App
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