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.

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

Add management for setting SOCIALACCOUNT_USER_GROUP #1001

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

wlorenzetti
Copy link
Member

@wlorenzetti wlorenzetti commented Dec 22, 2024

On Social Login action, the possibility to set an existing user group in the settings has been added when the user who logs in via social for the first time (Oauth2) is created.

local_settings.py:

...
...
SOCIALACCOUNT_USER_GROUP = '<existing_user_group>'

Related to : #994

@wlorenzetti wlorenzetti self-assigned this Dec 22, 2024
@wlorenzetti wlorenzetti added feature New feature or request backport to v.3.8.x labels Dec 22, 2024
@wlorenzetti wlorenzetti added this to the v3.9 milestone Dec 22, 2024
Copy link
Collaborator

@moovida moovida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@wlorenzetti wlorenzetti marked this pull request as ready for review December 23, 2024 07:55
@wlorenzetti wlorenzetti merged commit a38eabd into dev Dec 23, 2024
6 checks passed
@wlorenzetti wlorenzetti deleted the social_auth_group branch December 23, 2024 07:55
github-actions bot pushed a commit that referenced this pull request Dec 23, 2024
Co-authored-by: wlorenzetti <[email protected]>
(cherry picked from commit a38eabd)
wlorenzetti added a commit that referenced this pull request Dec 23, 2024
Co-authored-by: wlorenzetti <[email protected]>
(cherry picked from commit a38eabd)

Co-authored-by: Walter Lorenzetti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport to v.3.8.x feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants