Skip to content

Daniela and Xtina's Bonkers Video Store#17

Open
mystioreo wants to merge 80 commits into
Ada-C10:masterfrom
mystioreo:master
Open

Daniela and Xtina's Bonkers Video Store#17
mystioreo wants to merge 80 commits into
Ada-C10:masterfrom
mystioreo:master

Conversation

@mystioreo

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-C10/VideoStoreConsumer-API#17
For each person in your pair, describe something you learned from your partner during this project Daniela learned some CSS styling tricks from Christina. Christina learned a ways to manage passing props and working through managing state.
What was one area of React you gained more clarity on during this assignment? We got a better handle on managing state in different parts of a project as well as deciding which component should manage state.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content As far as we can tell we are displaying the same amount of content for each movie and the images are all the same size, so we didn't see this as a problem to solve.
Describe how you handled rentals with React We had a Rental Manager component that kept track of which customer and which movie were selected for the rental. The Rental Manager sent callback functions as props into the movie and customer components. When the create rental button is clicked, Rental Manager sends a POST request to the API to create the rental in the database.
Describe a DOM event your application handled We used the onClick event for selecting customers and movies.
Did you use any functional stateless components? What for? We chose to use classical components for every component.
Did you use any container components? What for? Customers is the container that holds the list of Customer components. Library is the container that holds all the Movies components.
Do you have any recommendations on how we could improve this project for the next cohort?
The project was good. It would be nice if deployment were more straightforward. More work with the Rails API would be good too.

mystioreo and others added 30 commits December 17, 2018 14:19
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.

1 participant