Dionisia - Edges - Inspiration-Board

[larachan15]

Inspiration Board

Congratulations! You're submitting your assignment!

Comprehension Questions

Question	Answer
Explain the steps in creating a new Card from the form.	When a user fills out the form and hits the submit button, this triggers the onFormSubmit event handler. It creates a newCard object. This is added to the API database by using the updated state as a parameter for the callback function addCard.
How did you learn how to use the API?	I used Postman and I also read the API documentation
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 to get the list of cards from the API. I am not sure I fully understand how this works though.
Explain the purpose of a Snapshot test.	It's to help us determine if our HTML we expect to render is rendered correctly. We can use it to compare the previous JSX that was rendered to the current JSX rendered.
What purpose does Enzyme serve in testing a React app?	Enzyme helps us create two types of rendering of our components: deep(renders the component and all its children) and shallow(renders the component, but not the children).

[droberts-sea]

Inspiration Board

What We're Looking For

Feature	Feedback
Core Requirements
Git hygiene	I would like to see much more frequent commits
Comprehension questions	yes
General
Card Component renders the data provided as props	yes
Board Component takes a URL and renders the list of Cards and passes in callback functions	yes
NewCardform Component is a controlled form and uses a callback function to return entered data to the parent component	yes
API
GET request made in componentDidMount	yes
DELETE request made in callback function	yes
POST request made in callback function passed to NewCardForm component.	yes
Snapshot testing	yes
Styling	yes
Overall	Good work overall. I've left a couple of inline comments below, but in general it's clear to me that the learning goals around integrating a React app with an API have been met. Keep up the hard work!

[droberts-sea]

I like that you've kept all the API interaction logic in one place - the callbacks are a little more complex, but I would say it makes the app as a whole much easier to comprehend. Whether or not you intended it, this is a great example of the container component pattern well-applied.

[droberts-sea]

Could this be a functional component?