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

Amber Lynn | Edges | Inspiration Board #43

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

Conversation

griffifam
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.
How did you learn how to use the API?
What function did you use to place the GET request from the API to get the list of cards? Why use that function?
Explain the purpose of a Snapshot test.
What purpose does Enzyme serve in testing a React app?

@droberts-sea
Copy link

Inspiration Board

What We're Looking For

Hi Amber Lynn! As we discussed in person:

  • This assignment is incomplete
  • It seems like a lot of the trouble is not necessarily your comprehension of React, since you were able to build various pieces of it as we sat together
  • Instead, where I see room for improvement is around debugging skills, planning / diagramming, and generally having a strong engineering workflow. Please be mindful of these skills as you move forward.

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