-
Notifications
You must be signed in to change notification settings - Fork 74
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
global: fix user register #30
global: fix user register #30
Conversation
# Save token if used is authenticated | ||
if current_user.is_authenticated: | ||
uid = current_user.get_id() | ||
# Save token if used is not anonymous (user exists but can be not active at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Save token if user is not anonymous ...
6edddfc
to
07eafa9
Compare
'OAUTHCLIENT_REMOTE_APPS'] | ||
[remote.name].get('remember', False)): | ||
# redirect the user after registration (which doesn't include the | ||
# activation), waiting user confirm his email |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waiting for user to confirm his email.
f2cdc1d
to
7a0da33
Compare
@@ -115,11 +117,15 @@ | |||
DEBUG=True, | |||
SECRET_KEY='TEST', | |||
SECURITY_PASSWORD_SALT='security-password-salt', | |||
SECURITY_LOGIN_WITHOUT_CONFIRMATION=False, | |||
# MAIL_SUPPRESS_SEND=False, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO we can remove this comment and TESTING
. Thanks
* Introduces the possibility to "ask for email confirmation" in case the provider does not return any user email (e.g. with orcid). Signed-off-by: Leonardo Rossi <leonardo.r@cern.ch>
7a0da33
to
2a20c48
Compare
the provider doesn't return any user email (e.g. with orcid).
Signed-off-by: Leonardo Rossi leonardo.r@cern.ch