Skip to content

Jete && Alex #17

Open
codesrobertson wants to merge 78 commits intoAda-C13:masterfrom
jetabajrami:master
Open

Jete && Alex #17
codesrobertson wants to merge 78 commits intoAda-C13:masterfrom
jetabajrami:master

Conversation

@codesrobertson
Copy link

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#8
For each person in your pair, describe something you learned from your partner during this project Different workflow styles.
What was one area of React you gained more clarity on during this assignment? JSX.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Card decks with different buttons.
Describe how you handled rentals with React Allowed for a rental to be made after a movie and a customer had been selected.
Describe a DOM event your application handled Buttons that trigger onClick events, for example line 23 in Movie.js.
Did you use any functional stateless components? What for? SearchedMovie allowed for the display of movies that matched a search entry.
Did you use any container components? What for? We used container components for the card decks.
Do you have any recommendations on how we could improve this project for the next cohort? Don't have this project outside of the React timeframe--if we'd done this project closer to the time we learned it, it would have reinforced the materials really nicely; as-is, we were scrambling to work with concepts that felt like we only glossed over in the first place. Overall, it's a fun project, but it needs to be in the React/JS portion.

Jetmire Bajrami and others added 30 commits June 22, 2020 19:37
jetabajrami and others added 30 commits June 24, 2020 18:19
added proptypes to all components
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