Skip to content

Lola & Nora#22

Open
thenora wants to merge 57 commits intoAda-C13:masterfrom
ubeninja77:master
Open

Lola & Nora#22
thenora wants to merge 57 commits intoAda-C13:masterfrom
ubeninja77:master

Conversation

@thenora
Copy link

@thenora thenora commented Jun 26, 2020

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C13/video-store-consumer-api#15
For each person in your pair, describe something you learned from your partner during this project We both agreed that we were using this project to re-live/learn React. We learned to be comfortable acknowledging if we were uncertain about something and asking for help. Working through the problem together.
What was one area of React you gained more clarity on during this assignment? We both gained more clarity in making API calls and React Router.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content The functions used to display them in movies and library were the same, and we used their pictures as an indicator.
Describe how you handled rentals with React We made functions for selecting a customer and a movie.
Describe a DOM event your application handled When you click on a movie, it triggers the onClick events
Did you use any functional stateless components? What for? Our display for the checkout process to show the selected movie / customer and checkout button was stateless.
Did you use any container components? What for? Our customer and library held their respective lists. App.js is container for components.
Do you have any recommendations on how we could improve this project for the next cohort? More information on react routing. Also, we understand this lesson was given to us now due to the change in timeline. But this lesson definitely should be done during React/JS time, or at least have had a TINY react project refresher before one this robust.

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