Skip to content

Time - Sharon & Emily#27

Open
sharonkeikei wants to merge 74 commits intoAda-C13:masterfrom
sharonkeikei:master
Open

Time - Sharon & Emily#27
sharonkeikei wants to merge 74 commits intoAda-C13:masterfrom
sharonkeikei:master

Conversation

@sharonkeikei
Copy link

@sharonkeikei sharonkeikei 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 https://github.com/Ada-C13/video-store-consumer-api/pull/20
For each person in your pair, describe something you learned from your partner during this project Sharon: Emily thoroughly explained the requirements to me helped me understand the big pictures of how this movie rental app works.
What was one area of React you gained more clarity on during this assignment? Sharon: I think this is a very good revision for react, we took the first and a half day just trying to study what we have learnt in the past.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We actually have two stateless components to have different thing rendering back, would like to improve the implementation in the future.
Describe how you handled rentals with React We have a useState hook for selecting the customer and movie and will call the make rental function to send the api post call to create a new rental if that two items are there.
Describe a DOM event your application handled We use event.target.value for the search bar in order to set it as the query for the api call to look for some specific movies.
Did you use any functional stateless components? What for? Our movie, customer and searchResult components are functional stateless, they are only there to be rendered by another component to show each individual customer / movie information thru the props.
Did you use any container components? What for? I think our library and Customers component are container components, they are there to just there to hold the other components like customer and movie.
Do you have any recommendations on how we could improve this project for the next cohort? It was a little bit harder to refresh the memory for React, maybe a couple more days would be nice, and also we would like to learn more about the deployment or maybe have some lectures on it.

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