Skip to content

Anna & Mona - Fire#7

Open
MonaRahmani wants to merge 75 commits into
Ada-C14:masterfrom
MonaRahmani:master
Open

Anna & Mona - Fire#7
MonaRahmani wants to merge 75 commits into
Ada-C14:masterfrom
MonaRahmani:master

Conversation

@MonaRahmani

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/pull/4
For each person in your pair, describe something you learned from your partner during this project We learned a lot about React and how to connect Rails backend to React frontend together. We also learned about other cool functions like setTimeOut and using components for popups, etc.
What was one area of React you gained more clarity on during this assignment? We learn more about how to pass props to other components. Also, we learn more about callbacks.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We used a Movie component that has the information we need for a single movie. Then we used this component to make the different lists.
Describe how you handled rentals with React First, we used postman to check if the route is working then we used axios to send post requests. Then we called the Setcustomer and the Setmovie to empty the state. Also, we used Settimeout method to remove the message.
Describe a DOM event your application handled We used a lot of onClick event listeners and handlers to create popups, send forms, etc.
Did you use any functional stateless components? What for? only the NavBar component does not have state.
Did you use any container components? What for? MovieLibrary component is a container for the MovieComponent...
Do you have any recommendations on how we could improve this project for the next cohort?

MonaRahmani and others added 30 commits January 18, 2021 16:15
listed all videos in the video library
worked on CustomerLis- we can see a list of all customers
MonaRahmani and others added 30 commits January 20, 2021 17:51
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