Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Octos - VideoStore - Brandy and Jamila #14

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

brandyaustinseattle
Copy link

@brandyaustinseattle brandyaustinseattle commented Jun 22, 2018

Video Store Consumer

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Paste a link to the PR for your API Ada-C9/VideoStoreConsumer-API#16
For each person in your pair, describe something you learned from your partner during this project. Brandy's answer...I learned that it would be beneficial if I slowed down and thought through decisions in greater depth before jumping right into the implementation. Jamila's answer...there is always and easier way.
What was one area of React you gained more clarity on during this assignment? Jamila..How to change state that is housed in a container component. Brandy...Passing of functions between classes.
Describe how you solved the problem of having lists of movies that look mostly the same but have different content Jamila..we used conditional logic to render the appropriate buttons for each list.
Describe how you handled rentals with React Rentals is a form that exists in App.js. Jamila...We could have made it state housed in the App.js that was sent to the ohter components as a callback function.
Describe a DOM event your application handled The DOM event was people checking out a movie
Did you use any functional stateless components? What for? We didn't use functional stateless components though we tried to do so with customers. We weren't able to implement it appropriately.
Did you use any container components? What for? The rental form in App.js functions similar to a container component.
Do you have any recommendations on how we could improve this project for the next cohort? Make it at a different time. It's hard to have a big/complicated project right before break.

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