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

Branches - Michaela #29

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

Conversation

michaela260
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 data is submitted on the NewCardForm, and then it is sent to Board via callback as a parameter for the addCard method. This callback updates the [cards] state in Board. Then, the allCards method in Board creates a Card with the data from the cards state.
What function did you use to make the GET request from the API to get the list of cards? Why use that function? I used componentDidMount() because I want the site to get data from the API as soon as the board component is created.
How do snapshot tests differ from unit tests? Snapshot tests only test whether the site has changed at all since your snapshot, they do not test whether a specific thing is the way you want it to be.
What purpose does Enzyme serve in testing a React app? I have never heard of Enzyme? Maybe you mean Jest? Jest is the testing framework that we use that allows us to run snapshot tests. Apparently Enzyme was developed by Airbnb to add more testing functionality specifically for React.

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