Skip to content

Commit 15a4e3a

Browse files
authored
Account: organize allauth settings (#9865)
- make email verification mandatory: users without a verified email will be asked to verify their address before being able to login - rename `ACTIVATION_DAYS` since it was a setting from a Django app we don't use anymore. It's called EMAIL_CONFIRMATION_EXPIRE_DAYS in Django Allauth. This PR requires other PRs in -ops to work as intended.
1 parent c445700 commit 15a4e3a

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

readthedocs/settings/base.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -710,11 +710,16 @@ def DOCKER_LIMITS(self):
710710
# All auth
711711
ACCOUNT_ADAPTER = 'readthedocs.core.adapters.AccountAdapter'
712712
ACCOUNT_EMAIL_REQUIRED = True
713+
714+
# Make email verification mandatory.
715+
# Users won't be able to login until they verify the email address.
713716
ACCOUNT_EMAIL_VERIFICATION = 'mandatory'
717+
714718
ACCOUNT_AUTHENTICATION_METHOD = 'username_email'
715-
ACCOUNT_ACTIVATION_DAYS = 7
719+
ACCOUNT_EMAIL_CONFIRMATION_EXPIRE_DAYS = 7
716720
SOCIALACCOUNT_AUTO_SIGNUP = False
717721
SOCIALACCOUNT_STORE_TOKENS = True
722+
718723
SOCIALACCOUNT_PROVIDERS = {
719724
'github': {
720725
'SCOPE': [

0 commit comments

Comments
 (0)