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

[fix] Admin actions require model permissions #210

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

pandafy
Copy link
Member

@pandafy pandafy commented Jun 18, 2024

Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

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

Please rebase on the latest master to fix the CI build.

Comment on lines 73 to 74
# TODO: Remove before merging
pip install --force-reinstall --no-cache-dir --no-deps --upgrade https://github.com/openwisp/openwisp-utils/tarball/admin-action-perm
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# TODO: Remove before merging
pip install --force-reinstall --no-cache-dir --no-deps --upgrade https://github.com/openwisp/openwisp-utils/tarball/admin-action-perm

@pandafy pandafy force-pushed the fix-admin-actions branch from 95786e8 to c059b64 Compare July 2, 2024 10:58
@coveralls
Copy link

Coverage Status

coverage: 99.351%. remained the same
when pulling c059b64 on fix-admin-actions
into 60a00a3 on master.

@nemesifier nemesifier merged commit 7d1fe36 into master Jul 2, 2024
20 checks passed
@nemesifier nemesifier deleted the fix-admin-actions branch July 2, 2024 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants