Skip to content

Roshni & Madeline #30

Open
made-line wants to merge 16 commits into
Ada-C14:masterfrom
made-line:master
Open

Roshni & Madeline #30
made-line wants to merge 16 commits into
Ada-C14:masterfrom
made-line:master

Conversation

@made-line

@made-line made-line commented Jan 22, 2021

Copy link
Copy Markdown

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