-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Introduce ActionPolicy authorization adapter #8708
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
base: master
Are you sure you want to change the base?
Conversation
References: - https://gist.github.com/amkisko/c704c1a6462d573dfa4820ae07d807a6 - actionpolicy.evilmartians.io
This commit introduces the ActionPolicy adapter for ActiveAdmin, allowing for flexible authorization management. The ActionPolicy gem is added to the Gemfile, and relevant documentation is updated to guide users on configuring the adapter. Additionally, new policy templates are included to demonstrate usage. Ref: https://actionpolicy.evilmartians.io
@tagliala @javierjulio Hey! Here's better PR for review and discussion. There is one critical thing to figure out is how to organize test templates as they are intersecting with Pundit. I suggested the option, but really I don't like it at this point, but it should work. Also I could not figure out yet what is missing on my machine for running rspec, tests are not checked. And sure before merging squashing required (so that you can edit commit message to the preferred format) or I can recreate it again. |
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.
Hello,
Thanks for this PR.
Also I could not figure out yet what is missing on my machine for running rspec, tests are not checked.
Maybe something related to bundler? I've seen the removal of ruby
, so I guess that build native extensions
is somehow not working.
Run specs with just CI=1 rake
, that will create the test app and proceed with all the specs
First comments:
- Rebase from baster
- Remove the change in the comment
- Please make sure that specs and linters pass locally
313 files inspected, 17 offenses detected, 17 offenses autocorrectable
@@ -415,10 +412,10 @@ GEM | |||
|
|||
PLATFORMS | |||
arm64-darwin | |||
ruby |
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.
Please remove this change
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.
Yeah, I saw this one, it was automatically modified by bundler locally.
This PR introduces a new ActiveAdmin::ActionPolicyAdapter to enable integration with the Action Policy authorization framework. The adapter implements both #authorized? and #scope_collection methods, mapping ActiveAdmin’s internal permissions to Action Policy’s rules and scopes.
ActiveAdmin supports a pluggable authorization system, but out of the box, it only provides adapters for popular frameworks like Pundit and CanCanCan. As more teams adopt Action Policy for its performance, flexibility, and first-class Rails support, it’s increasingly desirable to make ActiveAdmin compatible with it out of the box.
References: