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

Does an admin create a new User ? #103

Open
ankitjain28may opened this issue Nov 19, 2017 · 4 comments
Open

Does an admin create a new User ? #103

ankitjain28may opened this issue Nov 19, 2017 · 4 comments

Comments

@ankitjain28may
Copy link
Contributor

No description provided.

@itaditya
Copy link
Contributor

Yes, for testing purposes an admin is allowed to create user. This feature is added of line 63 in https://github.com/ncs-jss/plexus-17/blob/master/routes/user.route.js

@ankitjain28may
Copy link
Contributor Author

So, there is a bug in the code, as we are only checking whether the authId is there or not.. but when admin creates a new user... he can add only email and name so a new user is created but.. after creating a single user... admin wont be able to create more users as per the condition as the user with authId = undefined is already there.

@itaditya
Copy link
Contributor

Yeah, I knew about that, thanks for bringing this up. Can you suggest a possible patch for this.

@itaditya
Copy link
Contributor

Also, create an issue regarding the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants