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.

Sign up for GitHub

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

Jump to bottom

Log in after confirming email address #823

Merged
merged 1 commit into from Mar 14, 2019
Merged

Log in after confirming email address #823

merged 1 commit into from Mar 14, 2019

Conversation

symroe
Copy link
Member

@symroe symroe commented Mar 14, 2019

Fixes #793

From the docs:

ACCOUNT_LOGIN_ON_EMAIL_CONFIRMATION (=False)

The default behaviour is not log users in and to redirect them to ACCOUNT_EMAIL_CONFIRMATION_ANONYMOUS_REDIRECT_URL.

By changing this setting to True, users will automatically be logged in once they confirm their email address. Note however that this only works when confirming the email address immediately after signing up, assuming users didn’t close their browser or used some sort of private browsing mode.

@ghost ghost assigned symroe Mar 14, 2019
@ghost ghost added the 4 - Awaiting Review label Mar 14, 2019
@coveralls
Copy link

coveralls commented Mar 14, 2019 •
edited

Coverage Status

Coverage remained the same at 74.663% when pulling 8b7570b on 793-login-after-confirming-email-address into 1b290f0 on master.

@symroe symroe merged commit 90867bf into master Mar 14, 2019
@ghost ghost removed the 4 - Awaiting Review label Mar 14, 2019
@symroe symroe added this to Done in CandidateWiki Mar 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris48s chris48s chris48s approved these changes

Assignees

@symroe symroe

Labels
None yet
Projects
No open projects
CandidateWiki
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Log in after confirming email address
3 participants
@symroe @coveralls @chris48s