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

Users with allowed unconfirmed access can now log in successfully. #202

Merged
merged 1 commit into from
Jul 3, 2015

Conversation

colavitam
Copy link
Contributor

These changes allow unconfirmed users to log in, provided that the time is within the interval specified by allow_unconfirmed_access_for.

@lynndylanhurley
Copy link
Owner

Hi @colavitam - thank you for sending this PR! Sorry for my late response - I'll review ASAP!

@killerham
Copy link

Any news on this pull request?

@mooremo
Copy link

mooremo commented Jun 5, 2015

👍

@lynndylanhurley
Copy link
Owner

I'm sorry I've been crazy busy for the past few weeks. I'll look at this tonight.

@mooremo
Copy link

mooremo commented Jun 25, 2015

+1 for this PR again. I'm having to use a forked version of the gem until it respects this option.

@booleanbetrayal
Copy link
Collaborator

LGTM!

@booleanbetrayal
Copy link
Collaborator

@colavitam - can you please resolve the conflicts (and preferably squash commits) against master?

@colavitam
Copy link
Contributor Author

@booleanbetrayal: Should be all set!

@booleanbetrayal
Copy link
Collaborator

Thanks!

booleanbetrayal added a commit that referenced this pull request Jul 3, 2015
Users with allowed unconfirmed access can now log in successfully.
@booleanbetrayal booleanbetrayal merged commit 7919666 into lynndylanhurley:master Jul 3, 2015
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.

5 participants