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

Add after_login_lock callback #236

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

y-yagi
Copy link

@y-yagi y-yagi commented May 7, 2020

This patch added a callback that execute only when user locked. This callback useful for run something actions(e.g. send notifications to admin) when user locked.

Please ensure your pull request includes the following:

  • Description of changes
  • Update to CHANGELOG.md with short description and link to pull request
  • Changes have related RSpec tests that ensure functionality does not break

This patch added a callback that execute only when user locked.
This callback useful for run something actions(e.g. send notifications
to admin) when user locked.
@y-yagi
Copy link
Author

y-yagi commented Feb 21, 2021

Any feedback?

@joshbuker joshbuker added the to be implemented in v1 This issue or pull request will be resolved in the v1 rework, but has not yet been completed. label May 27, 2021
@joshbuker
Copy link
Member

@y-yagi Thanks Yagi, I'll get this into v1. Are there any other callbacks that you've found yourself wishing Sorcery provided?

@y-yagi
Copy link
Author

y-yagi commented May 28, 2021

Thanks for the reply! This is the only callback that we need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be implemented in v1 This issue or pull request will be resolved in the v1 rework, but has not yet been completed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants