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

React JS basic assignment submitted by Neelam Verma #9

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

Conversation

neelamv222
Copy link

@neelamv222 neelamv222 commented Jul 28, 2017

Hello Team,

I have developed user list CRUD application. Please share your feedback.

@akshaynaik404
Copy link

@neelamv222 Your PR title is not very informative. Could you update it, so that it will be easy for reviewers to go through it.

@neelamv222 neelamv222 changed the title code changes inital commit. React JS basic assignment submitted by Neelam Verma Jul 29, 2017
@neelamv222
Copy link
Author

@akshaynaik404 Updated PR Title. Let me know the feedback of assignment.

@akshaynaik404
Copy link

I am not a reviewer. A participant just like you. :-)

*
**/
compareData() {
let dublipcateData = this.state.userList;

Choose a reason for hiding this comment

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

spelling mistake for duplicateData.

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.

None yet

4 participants