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

Leaves - Dominique #20

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

Leaves - Dominique #20

wants to merge 3 commits into from

Conversation

dtaylor73
Copy link

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? the values of each field in the form are assigned to the state of the NewCardForm component using the 'onInputChange' function. The 'onSubmitHandler' function passes the updated values in the state to the 'addCardCallback' function that lives in the Board component. The addCard function in the Board component makes the post request and updates the list of cards in Board's state. The new card is seen on the screen when 'makeCollection' renders the new list of cards.
What function did you use to make the GET request from the API to get the list of cards? Why use that function? componentDidMount(). I used this to make sure the component was fully rendered before making the get request. If i didn't use it, some cards could be loaded and other cards wouldn't be loaded.
How do snapshot tests differ from unit tests? Snapshot tests will check for changes during rendering. Snapshot tests are a very useful tool whenever you want to make sure your UI does not change unexpectedly.
What purpose does Enzyme serve in testing a React app? Enzyme is a JavaScript Testing utility for React that makes it easier to test your React Components' output.

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