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

Mary Ampers #6

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

Mary Ampers #6

wants to merge 5 commits into from

Conversation

mmlamkin
Copy link

@mmlamkin mmlamkin 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 passed in the selected madlib data as a prop to the form and then use map to return the correct input boxes for each word depending on the madlib
What area of React did you gain more clarity on in this assignment? I have a little more clarity on event handlers, but I'm still feeling pretty lost in general on this topic!!
What are of React do you still have outstanding questions on after this assignment? What can you put in the render function and what needs to be outside of that, types of functions to use ie arrow functions, 'this', 'event', 'target'
Do you have any recommendations on how we could improve this project for the next cohort? I think spending some more time on arrow functions would have helped me understand react better as everything seems to be an arrow function, as well as what this is in an particular moment.

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