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

Roshni & Madeline #30

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

Roshni & Madeline #30

wants to merge 16 commits into from

Conversation

made-line
Copy link

@made-line made-line 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#14
For each person in your pair, describe something you learned from your partner during this project Roshni: I learned more about props and how the data flows from one component to another. Madeline: I learned about changing a class component to a functional one and the importance of importing things.
What was one area of React you gained more clarity on during this assignment? We learned about the React Router, how to work with APIs in React, and the useEffect hook.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Assigning each movie within the list a key (external_id) allowed us to uniquely identify each movie.
Describe how you handled rentals with React We used state to keep track of the selected customer and video and were then able to checkout once both were selected by making a post request to the Rails API. We also needed to include the due date.
Describe a DOM event your application handled A DOM event our application handled was the adding of a video to the library. When the user clicks on the add to library button, the onClick function is triggered which uses the add video callback function we wrote to update our library.
Did you use any functional stateless components? What for? Yes, our Home component is stateless. We added this component in for display purposes.
Did you use any container components? What for? CustomerList and Library can be thought of as container components. They contain the lists of the individual customer and video components, respectively.
Do you have any recommendations on how we could improve this project for the next cohort? Having more lessons and an additional day to work on the project would have been very helpful.

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