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

allauth.account.middleware.AccountMiddleware must be added to settings.MIDDLEWARE in the demo project #555

Closed
daniel-unyi-42 opened this issue Oct 5, 2023 · 5 comments · Fixed by #561
Closed

allauth.account.middleware.AccountMiddleware must be added to settings.MIDDLEWARE in the demo project #555

daniel-unyi-42 opened this issue Oct 5, 2023 · 5 comments · Fixed by #561

Comments

@daniel-unyi-42
Copy link

It doesn't work otherwise.
@RonaldinhoL
Copy link

same

@RonaldinhoL
Copy link

nvm, just add allauth.account.middleware.AccountMiddleware to middleware

@ozgurturkiye
Copy link

Does order matter when adding "middleware"?

mweinelt added a commit to mweinelt/dj-rest-auth that referenced this issue Oct 23, 2023
@mweinelt
Install allacccount middleware in test settings
be0cf53 
Closes: iMerica#555
@mweinelt mweinelt mentioned this issue Oct 23, 2023
Merged
@mweinelt
Copy link
Contributor

Does order matter when adding "middleware"?

https://github.com/pennersr/django-allauth/blob/main/allauth/account/apps.py#L13-L17

The test here does not care for the order.

mweinelt added a commit to mweinelt/dj-rest-auth that referenced this issue Oct 23, 2023
@mweinelt
Install allacccount middleware in test settings
09b3b1a 
and upgrade django-allauth past 0.56.0.

Closes: iMerica#555
mweinelt added a commit to mweinelt/dj-rest-auth that referenced this issue Oct 23, 2023
@mweinelt
Install allacccount middleware in test settings
a1cc505 
and upgrade django-allauth past 0.56.0.

Closes: iMerica#555
iMerica pushed a commit that referenced this issue Oct 27, 2023
@mweinelt
Install allacccount middleware in test settings (#561)
8aa8b9a 
and upgrade django-allauth past 0.56.0.

Closes: #555
@colinkingswood
Copy link

This ought to be added to the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Install allacccount middleware in test settings mweinelt/dj-rest-auth
5 participants
@mweinelt @RonaldinhoL @colinkingswood @ozgurturkiye @daniel-unyi-42