-
Notifications
You must be signed in to change notification settings - Fork 8
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
Using permissions table to store permissions of roles. #1
Comments
Hi @dudzio12, Thank you for this contribution. Do you really need to store the authorization rules in a database ? Can you explain me a bit more your need of storing permissions in a database, please? Cheers, |
I'm building ERP-kind system. For adminstrator role I've got in my "permissions" table: So I've build this with bin above. Now for the final part: I've got in backoffice of my app: So basicly I can choose what groups of users are allow or deny to do some actions. I think I explain it clearly. I suppose I can inserting into config file, but it would be more messy I think. Thanks for your work! :) |
Thank you, now I understand your need. Are you sure this line is useful ? Maybe I'll add your feature in the next minor version of Authority-Controller. But I don't plan to do it yet. Cheers, |
Looking good :) Yes, in specific way - as I said I'm building ERP system. As I mentioned before, I found somewhere in Laravel forum thread about the same problem as I resolved, unfurtunatelly it is off now, so I just paste it here, if you think there is better place to paste it - feel free to use it. Thanks for conversation :) |
Hi!
Few people have problem with using permissions table to manage permissions on roles.
By default permissions table is using by User model, so in my opinion is a little bit overpowered, I mean - more often we need to set permissions for roles, not for users itself, so here it is what I've done to accomplished that:
http://laravel.io/bin/5WMxJ
With this we can easily manage users, roles and permissions from office backend.
Hope will be helpful for someone :)
The text was updated successfully, but these errors were encountered: