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

Logg out inactive users, and redirected them to the login page #683

Open
wants to merge 2 commits into
base: 5.0.x
Choose a base branch
from

Conversation

justindh
Copy link

Currently a logged in but deactivated user can use seat and all of its plugins with their current permissions set. An inactive user that tries to login will receive an error. This change forces any user that is inactive, to be logged out and redirected to the login page.

Akov added 2 commits July 13, 2024 16:39
Logged out inactive users, and redirected them to the login page
@tehraven
Copy link
Contributor

I like this.

Seems to be some formatting issues, can you run the code formatter so you don't run into that issue with maintainers get around to reviewing this? (EX if ($user->active == true){ without the space before bracket)

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

Successfully merging this pull request may close these issues.

2 participants