Skip to content

Contact List submission # 1#80

Open
cwjohnson7 wants to merge 8 commits intoprojectshft:mainfrom
cwjohnson7:main
Open

Contact List submission # 1#80
cwjohnson7 wants to merge 8 commits intoprojectshft:mainfrom
cwjohnson7:main

Conversation

@cwjohnson7
Copy link
Copy Markdown

No description provided.

</div>
</div>
)
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Watch your indentation here

history.push(`/contacts/${contact.id}`)
}
return (

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This space isn't necessary

}
addContact(newContact);
contactsPage();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This space isn't necessary

import axios from "axios";


class App extends Component {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Interesting to do a class component here when you use functional components everywhere else

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