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

Auto Login filter #248

Open
designermonkey opened this issue Jun 23, 2014 · 0 comments
Open

Auto Login filter #248

designermonkey opened this issue Jun 23, 2014 · 0 comments

Comments

@designermonkey
Copy link
Member

The auto login filter is wrapped with logic to disallow it's use with an Activation field in the Members section.

This logic should be removed, as the Activation field itself will determine whether a member can log in or not.

Suggestion: We promote auto login to use it's own logic instead of piggybacking on the Update Password Login filter. Reason being, that there is no result appended to the event messages when this filter fails, for example, if another filter prevents the rest from running. Unless I'm missing something and this is expected behaviour?

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

No branches or pull requests

1 participant