Skip to content

Madeline Olsen #24

Open
madelineolsen wants to merge 5 commits intoAustinCodingAcademy:masterfrom
madelineolsen:master
Open

Madeline Olsen #24
madelineolsen wants to merge 5 commits intoAustinCodingAcademy:masterfrom
madelineolsen:master

Conversation

@madelineolsen
Copy link

advanced contact list starter... apologies, i didnt realize i hadnt done this

Copy link
Contributor

@canaandavis canaandavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work overall.

You didn't really hit any of the extensions, though I see where you are attempting to start the selected contacts one.

You also didn't add the ability to delete a contact. I would like to see you add that ability along with the selected contacts at a minimum.

How are things progressing for you, do you feel like you understand the work you've done here?

}
}

handleNameChange(event) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See if you can reduce the three handle functions into one, there is a blurb in the lesson about it!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm struggling greatly with react. To the point that I kind of don't know what questions to ask. I've begun studying more outside of class this week but I'm still very behind as far as understanding. I plan to spend a lot of this next week just trying to get the basics down. This is why I couldn't come up with any of the extensions. Promise I tried! Will continue trying.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, well lets chat tonight, and see if we can't get you on the right path!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants