Skip to content

Charlotte and Vicki #25

Open
vwhwang wants to merge 77 commits intoAda-C13:masterfrom
vwhwang:master
Open

Charlotte and Vicki #25
vwhwang wants to merge 77 commits intoAda-C13:masterfrom
vwhwang:master

Conversation

@vwhwang
Copy link

@vwhwang vwhwang 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#17
For each person in your pair, describe something you learned from your partner during this project Charlotte: Vicki is great at styling. Vicki: Charlotte is good navigating while pair coding.
What was one area of React you gained more clarity on during this assignment? Vicki: How to pass props between components. Charlotte: How front end request data from backend.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content We did not notice this issue.
Describe how you handled rentals with React we used post for checkout action in front end.
Describe a DOM event your application handled we used state for selected movie and customer on app.js
Did you use any functional stateless components? What for? yes, we used movie to represent individual buttons of movies.
Did you use any container components? What for? yes, Movielist contains movie.
Do you have any recommendations on how we could improve this project for the next cohort? Have this project closer to when react was taught.
links to deployment https://vwhwang.github.io/video-store-consumer/#/

FreeMonkey19 and others added 30 commits June 24, 2020 21:30
added npm moment library for dateformating
added movie and customer to be side by side
Merge pull request #12 from vwhwang/master
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