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

Ports - Laneia #45

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

Ports - Laneia #45

wants to merge 4 commits into from

Conversation

laneia
Copy link

@laneia laneia commented Jun 25, 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 collects the data and sends it to the addCardCallback function in Board.js which posts the card via the API.
What function did you use to make the GET request from the API to get the list of cards? Why use that function? It's inside componentDidMount so that the component is mounted prior to the get request.
How do snapshot tests differ from unit tests? Snapshot tests are testing component rendering versus other functionality.
What purpose does Enzyme serve in testing a React app? It makes testing easier to write/read and more maintainable.

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