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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can block a user #1705

Merged
merged 3 commits into from Aug 20, 2018
Merged

Can block a user #1705

merged 3 commits into from Aug 20, 2018

Conversation

lauriejim
Copy link
Contributor

My PR is a: 馃殌 New feature
Main update on the: Plugin

Let you block a user. He will can login and use existing JWT
Related and fix #1216

@lauriejim lauriejim added pr: 馃殌 New feature source: core:content-manager Source is core/content-manager package labels Aug 6, 2018
@lauriejim lauriejim self-assigned this Aug 6, 2018
@lauriejim lauriejim requested a review from soupette August 6, 2018 15:50
Copy link
Contributor

@soupette soupette left a comment

Choose a reason for hiding this comment

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

LGTM

@Mr-Black-Dahlia
Copy link

Let you block a user. He will can login and use existing JWT

So this will essentially block login access for a user? Will this log that user out or invalidate their token if they are currently logged in? Just wanted a little clarification. Thank you!

@lauriejim
Copy link
Contributor Author

@indiecore-fm this will also block any actions of this user https://github.com/strapi/strapi/pull/1705/files#diff-5a61899a2abde7c09554cb2fe47cdbcdR27

@lauriejim lauriejim added this to In progress in 3.0.0-alpha.14 via automation Aug 20, 2018
@lauriejim lauriejim added this to the 3.0.0-alpha.13.2 milestone Aug 20, 2018
@lauriejim lauriejim merged commit 0f4b43e into master Aug 20, 2018
3.0.0-alpha.14 automation moved this from In progress to Done Aug 20, 2018
@lauriejim lauriejim deleted the block-user branch August 20, 2018 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
source: core:content-manager Source is core/content-manager package
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Enable/Disable (Block) field for Users
3 participants