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

Karinna Iniguez - MadLib - Octos #8

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

Conversation

karinnainiguez
Copy link

@karinnainiguez karinnainiguez commented Jun 1, 2018

MadLib

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe how you approached tying each input box to a specific work in the MadLib. I called the map function on a word collection that was passed in as a prop. That way, I had access to the word's label and key in order to create the JSX to create each input box.
What area of React did you gain more clarity on in this assignment? I got more clarity on callback functions being given to children components through props and then calling those functions later from the child component.
What are of React do you still have outstanding questions on after this assignment? I still need a little bit of clarity on what should be handled by a parent component and what should be handled by the child component itself.
Do you have any recommendations on how we could improve this project for the next cohort? Maybe have a larger project where we create the Story and App sections together as a class or something.

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