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

Sockets - Karla #21

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

Sockets - Karla #21

wants to merge 6 commits into from

Conversation

kguadron
Copy link

@kguadron kguadron commented Jun 24, 2019

Inspiration Board

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What is the flow of data from input on the form to being a new card on the board? NewCardForm has its own state which is the content the user inputs into the form from the browser. This data is passed to a callback function in Board which facilitates sending an axios request to the API and updating its state accordingly.
What function did you use to make the GET request from the API to get the list of cards? Why use that function? componentDidMount so that the API request is made after the component is mounted/rendered
How do snapshot tests differ from unit tests? Snapshot tests are testing views
What purpose does Enzyme serve in testing a React app? We use it to make sure components rendered the way we expect them to

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