Sigrid Benezra - Edges - Inspiration Board

[sdbenezra]

Inspiration Board

Congratulations! You're submitting your assignment!

Comprehension Questions

Question	Answer
Explain the steps in creating a new Card from the form.	When you type your message or choose an emoji into the form, the form state updates. When you hit the Add Message button the state is saved in a variable that is passed to the addCardCallback passed through props from the Board.js file. That callback function does an axios post and also updates the Board's state which renders all the cards again including the new one.
How did you learn how to use the API?	I reviewed the docs on Github and tried out the get, post, and delete methods with Postman.
What function did you use to place the GET request from the API to get the list of cards? Why use that function?	I used componentDidMount(). It's run once when the board component is mounted, and we only need to retrieve the cards from the API once when we first start our app. Subsequent changes in cards are handled by the functions that post and delete cards.
Explain the purpose of a Snapshot test.	The purpose of the snapshot test is to check if your components are rendering properly.
What purpose does Enzyme serve in testing a React app?	Enzyme provides a function called shallow that allows us to limit the depth of rendering to one level so we can do basic testing of a parent without having to worry about child components causing errors.

[tildeee]

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	mostly missing tests
Styling	x
Overall

Nicely done on this Sigrid! I have a few comments on code style, and you're largely missing snapshot testing, but otherwise it's a good submission. Good work

[tildeee]

Hm, it'd be nice if you could have used the props boardName and url with this too

[tildeee]

It might make more sense to define this function deleteCard outside of render but still within Card. Then, to reference it, you would use this.deleteCard

[tildeee]

Would expect index, id, text, and emoji to be declared in propTypes