Skip to content

Conversation

@marikoja
Copy link

Inspiration Board

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Explain the steps in creating a new Card from the form. Users input a message and or an emoji name into the form fields, as the user types the onFieldChange function is called. When the form is submitted the addCardCallback is called to pass the props in via a function.
How did you learn how to use the API? I learned how to use the API by using postman in conjunction with console.log and the docs.
What function did you use to place the GET request from the API to get the list of cards? Why use that function? The componentDidMount was used to signal that the component should be re-rendered when the API is called.
Explain the purpose of a Snapshot test. Snapshot testing allows us to test for what the DOM should look like.
What purpose does Enzyme serve in testing a React app? Enzyme lets the developer use jest for testing and makes both shallow and deep mount testing possible.

@tildeee
Copy link

tildeee commented Jul 9, 2018

Inspiration Board

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene x
Comprehension questions x
General
Card Component renders the data provided as props x
Board Component takes a URL and renders the list of Cards and passes in callback functions x
NewCardform Component is a controlled form and uses a callback function to return entered data to the parent component x
API
GET request made in componentDidMount x
DELETE request made in callback function x
POST request made in callback function passed to NewCardForm component. x
Snapshot testing x
Styling x
Overall

Mariko!!

Great job on this project :) The code looks good and neat and clean.

Also, I like the touch of adding the status code!

Overall, good work!

(Also lol, I had this tab sitting on computer for days and didn't press submit. Sorry!)

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.

2 participants