-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Backport 3.2.x][Fixes #7355] Introduce the adoption of groups to define users permissions, and define add_resource permission for users #7451
Conversation
This pull request introduces 1 alert when merging 7f76821 into 6495e9f - view on LGTM.com new alerts:
|
@marthamareal LGTM complains about a new unused import introduced with this PR, please, take a look. Thanks. |
627e602
to
3fe0e32
Compare
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.
@marthamareal let's use lowercase for GeoNode permission names, so please change ADD_RESOURCE
to add_resource
.
- Assign users to contributors group by default
3fe0e32
to
d196226
Compare
contributors
with permissionbase_addresourcebase
.get_user_perms
functionChecklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.