-
-
Notifications
You must be signed in to change notification settings - Fork 638
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
Define Permissions on Specific Resource Attributes #420
Comments
+1 |
+1 |
1 similar comment
+1 |
This would be incredibly useful. |
I've been looking at the code for CanCanCan vs. CanCan 2.0 for this feature and I think it is doable. I'd be willing to work it into a pull request. If I got this working, how soon would I be able to get it merged into a release? |
@coorasse I have some ideas about how to approach this, but I would love to talk it over with you. What is the best way to contact you? |
Hello @phaedryx , I would be happy to introduce such a feature in cancancan 3.0 since is something that the community would like to see and also Ryan thought about it already seven years ago. |
I've got the pull request done: #474 Kick the tires and tell me what you think. |
This is addressed in the branch |
@coorasse Hi! is there any documentation of this feature? I was not able to find it. Just I found some specs: cancancan/spec/cancan/ability_spec.rb Line 566 in f9372fc
I tried it and I was expecting that |
Hey, I faced the same problem as you, will be glad for any help, how did you solve this? |
@pjmartorell @showthreadx I don't see any changes to load_and_authorized_resource in the #474. It would be quite a bad idea actually. But this PR introduces |
This is more of a feature request, copied from original project's 2.0 branch:
Considering Ryan's comment on the ticket here, it might be worth looking at that implementation and just export it the current project?
The text was updated successfully, but these errors were encountered: