Skip to content
This repository was archived by the owner on Feb 2, 2019. It is now read-only.

Conversation

@Sandeepvarri
Copy link
Contributor

Only implemented UI of signup component

@prakritidev
Copy link
Contributor

@devilcrucifier @mr-woot can you review this? and also check for lint, travis will not build if there is something wrong

Copy link
Member

@mr-woot mr-woot left a comment

Choose a reason for hiding this comment

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

Remove comments

@prakritidev
Copy link
Contributor

@mr-woot is console.log() best practice ? you told me it's not. I think @Sandeepvarri you have to remove those too.

Lemme know if I'm wrong.

@Sandeepvarri
Copy link
Contributor Author

@prakritidev I will remove while intigrating with google, github and linkedin.
I just kept for checking functionality.

@prakritidev
Copy link
Contributor

@prakritidev I will remove while intigrating with google, github and linkedin.
I just kept for checking functionality.

Ok, then do not merge into develop now. When you implement the full implementation, then request for PR on develop . Till then work on your feature branch only.

@Sandeepvarri
Copy link
Contributor Author

Okay

@prakritidev
Copy link
Contributor

@devilcrucifier checkout the changes, review and merge this branch.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants