-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
[deps] Update django-allauth requirement from ~=0.54.0 to ~=64.1.0 #403
[deps] Update django-allauth requirement from ~=0.54.0 to ~=64.1.0 #403
Conversation
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.
@pandafy can you add an OR run tests without the parallel flag to the test line? I am still not sure why some times errors are shown correctly even in parallel mode and some times not.
a759675
to
a8f5e98
Compare
Updates the requirements on [django-allauth](https://github.com/pennersr/django-allauth) to permit the latest version. - [Changelog](https://github.com/pennersr/django-allauth/blob/main/ChangeLog.rst) - [Commits](pennersr/django-allauth@0.54.0...64.1.0) --- updated-dependencies: - dependency-name: django-allauth dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
7ab5aaf
to
c748586
Compare
django-allauth>0.63.6 dropped support for Django 3.2
c748586
to
9cfdf85
Compare
data = {'email': '[email protected]', 'primary': True} | ||
with self.assertNumQueries(7): | ||
data = {'email': '[email protected]'} | ||
expected_queries = 7 if django.VERSION < (4, 0) else 9 |
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.
The extra queries are added because of validation of constraints.
@@ -1,6 +1,6 @@ | |||
django-organizations~=2.4.1 | |||
django-extensions~=3.2.3 | |||
django-allauth~=0.54.0 | |||
django-allauth~=0.63.6 |
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.
>=64
drops support for Django 3.2
@@ -67,6 +67,7 @@ | |||
'django.middleware.common.CommonMiddleware', | |||
'django.middleware.csrf.CsrfViewMiddleware', | |||
'django.contrib.auth.middleware.AuthenticationMiddleware', | |||
'allauth.account.middleware.AccountMiddleware', |
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.
We will need to port this change to ansible-openwisp2 after merging this PR.
@nemesifier there are two UI changes in the new version of django-allauth: The new version adds help text to the new password field in password change/reset form. The help text shows requirements from the password validators. It also adds password reset URL in help text for password field in the login form. |
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.
@nemesifier there are two UI changes in the new version of django-allauth:
The new version adds help text to the new password field in password change/reset form. The help text shows requirements from the password validators.
I would adjust text size and spacing here: the text size could be consistent with the rest of the page and there shouldn't be as much space below.
It also adds password reset URL in help text for password field in the login form. Shall we hide this URL? Or, shall we remove the button?
I think we can hide it since it's redundant with the button below.
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.
LGTM. Please remember to send a PR to pin openwisp-utils.
Updates the requirements on django-allauth to permit the latest version.
Changelog
Sourced from django-allauth's changelog.
... (truncated)
Commits
d6e07b2
feat: Django 5.1b7eaefd
chore: Release 64.1.04b13c66
feat(mfa):AuthenticateView
to use form class from settingbd3ae52
chore(i18n): Sync translations63bc165
feat(account): Login timeout4303cbe
fix(headless): Logout should remove partial login20d1334
chore(mfa): add tags to submit buttonc970b33
tests(socialaccount): Test storing tokens vs email authentication907ffe6
chore(i18n): korean translationsef8743c
fix(socialaccount): OAuth 1.0 vs storing tokens without app instanceDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Blockers