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.

Sign up for GitHub

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

Jump to bottom

Deactivate user #48

Closed
TimMikeladze opened this issue Feb 4, 2017 · 5 comments
Closed

Deactivate user #48

TimMikeladze opened this issue Feb 4, 2017 · 5 comments
Assignees
@TimMikeladze
Labels
enhancement help wanted

Comments

@TimMikeladze
Copy link
Member

No description provided.

@TimMikeladze TimMikeladze self-assigned this Feb 4, 2017
@davidyaha
Copy link
Member

davidyaha commented Mar 8, 2018 •
edited
Loading

I think accounts.js should never be able to remove a user. Lets settle for a deactivate?
What do you think @TimMikeladze ?

Aetherall pushed a commit that referenced this issue Mar 11, 2018
@pradel @Aetherall
Merge pull request #48 from accounts-js/renovate/ts-jest-22.x
94b37d1 
Update dependency ts-jest to v22
Aetherall pushed a commit that referenced this issue Mar 13, 2018
@pradel @Aetherall
Merge pull request #48 from accounts-js/renovate/ts-jest-22.x
cec1949 
Update dependency ts-jest to v22
@pradel pradel changed the title removeUser Deactivate user Jul 12, 2018
@pradel
Copy link
Member

pradel commented Oct 11, 2018

What should be the name of the field? blocked, deactivated, other ...?

Steps to do:

  • create the deactivateUser and activateUser functions for the database adapters.
  • Add the field to the user object as false when we create the user.
  • create a deactivateUser | blockUser function on accounts/server.
  • create a activateUser | unBlockUser function on accounts/server.
  • check during the authentication process if the blocked | deactivated field on the user object is not true.

@lorensr
Copy link
Contributor

lorensr commented Oct 11, 2018

deactivateUser/activateUser and a user field: deactivated: true. (absense of field means user is active)

@pradel pradel mentioned this issue Oct 14, 2018
Merged
2 tasks
@pradel
Copy link
Member

pradel commented Oct 14, 2018

Pull request ready for this issue see #480

pradel added a commit that referenced this issue Oct 15, 2018
@pradel
[@accounts/server] Deactivate user (#480)
ccd1312 
Close #48

- [x] Logic
- [x] Tests
@acomito
Copy link

acomito commented Dec 31, 2020

Just used this feature! thanks

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

@TimMikeladze TimMikeladze

Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@lorensr @TimMikeladze @davidyaha @pradel @acomito