Resolve "Let apps add extra permissions to models"
All threads resolved!
All threads resolved!
Closes #273 (closed)
Edited by Hangzhi Yu
Merge request reports
Activity
added part::backend prio::1 labels
added 1 commit
- affc3b1c - Add method to inject permissions to ExtensibleModels dynamically
assigned to @hansegucker
- Resolved by Jonathan Weth
assigned to @nik and unassigned @hansegucker
added 1 commit
- 57956aac - Add method to inject permissions to ExtensibleModels dynamically
- Resolved by Jonathan Weth
aleksis/core/util/apps.py:211:1: W391 blank line at end of file aleksis/core/util/apps.py:211:1: W293 blank line contains whitespace
Not lint-clean :)
added 3 commits
-
7da8a589...3ffe64e2 - 2 commits from branch
master
- 626bfadf - Merge branch 'master' into 273-let-apps-add-extra-permissions-to-models
-
7da8a589...3ffe64e2 - 2 commits from branch
This approach seems to completely bypass the regular permission creation in Django. Even if it looks correct now, it carries the risk of breaking if permission management in Django changes. In essence, we get a part of the Django coe as a code copy we have to maintain.
If you have considered these issues and have arguments against it, please describe those and why you think the code is stable.
added 22 commits
-
626bfadf...9d7bbf6a - 21 commits from branch
master
- 97d76f59 - Merge branch 'master' into 273-let-apps-add-extra-permissions-to-models
-
626bfadf...9d7bbf6a - 21 commits from branch
mentioned in commit 1d045dfb
Please register or sign in to reply