Skip to content

Conversation

@RX7RUSS
Copy link

@RX7RUSS RX7RUSS commented Apr 18, 2017

No description provided.

@RX7RUSS RX7RUSS changed the title russ.savage.atx.advanced1 russ.savage.atx.advanced.1 Apr 18, 2017
Copy link

@ivome ivome left a comment

Choose a reason for hiding this comment

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

That looks great! Try to always create functional components, instead of the class based components, when you don't need the extended react functionality (which we will introduce later)...

}

App.propTypes = {
arrayOfUsers: PropTypes.array
Copy link

Choose a reason for hiding this comment

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

Missing isRequired

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