Skip to content

Advanced Contact List Lessons #25

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

Advanced Contact List Lessons #25
zekeg wants to merge 5 commits intoAustinCodingAcademy:masterfrom
zekeg:master

Conversation

@zekeg
Copy link

@zekeg zekeg commented Feb 7, 2017

No description provided.

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.

Hey good stuff overall.

You didn't really hit the extensions / add ability to delete the contact.

But I know you had some issues and it took a lot of time.

I would like to see you go back and continue to work on this.

Based on your comments it seems like your starting to understand how things are flowing in the app, thats a good sign.

Just keep up the work and let me know when you update this.

import React, { Component } from "react";

//first I gotta npm install axios and then import it
//WHICH WAS A HUGE PAIN ON WINDOWS 10 AND I WASTED MY LIFE
Copy link
Contributor

Choose a reason for hiding this comment

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

Love this comment

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