-
Notifications
You must be signed in to change notification settings - Fork 226
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
feat: Add User Manager #393
feat: Add User Manager #393
Conversation
15918ac
to
427c4bc
Compare
rename things to User Account Manager. |
@jeevithakannan2 could you move these scripts to utils? + rename it to user account manager. whats user manager? |
It must be under utility it is not an application or system setup. And it should've been user account manager not user manager |
6a13f63
to
b64444e
Compare
that's exactly what i asked u to do. |
b64444e
to
4b1fc28
Compare
@jeevithakannan2 Please unresolve those reviews, they are not resolved in the slightest. |
They are resolved |
@jeevithakannan2 Check the comment I added in the review. I explained it more. |
Added them like you wanted. not going to ague over this. Wasted my time on this crap |
Don't ask to me squash the commits. I'm not going to do that !! Not going to waste my time on this again |
@jeevithakannan2 we reviewers ask you to do certain things so that the script follows the uniformity as well as is free from bugs which could happen by not calling those functions. now squashing your commits is necessary because chris wont accept it when u have so many commits on such a small pr. if you dk how to, we'll teach you. dont worry. |
@jeevithakannan2 open your fork in vs code. go to the respective branch where this is present. then, in terminal type |
Pull Request
User manager to create / modify users
Type of Change
Description
Add a user manager script that can do the following
Create a new user
Delete a user
Modify a user
Also added docs for utilities
Testing
Checklist