search

rappasoft / vault

Roles & Permissions for the Laravel 5 Framework
MIT License
103 stars 10 forks source link

Unable to deselect Administrator Role in Permission #5

Closed shailesh91 closed 9 years ago

shailesh91 commented 9 years ago

Hi, While trying out the package I noticed that while creating a new permission, I was unable to deselect the Administrator Role. Shailesh

rappasoft commented 9 years ago

Yes, my idea was the administrator should have all permissions. If you want to change it, publish the views and edit the view file to get rid of the readonly attribute.

shailesh91 commented 9 years ago

Hmm. Thanks, I will do that!

rappasoft commented 9 years ago

Maybe i'll make it a config option in the next release.

shailesh91 commented 9 years ago

That will be a good addition. Another thing, the package requires atleast one permission. Is that something on purpose?

rappasoft commented 9 years ago

It doesnt, if you don't see a delete box, edit the permission and uncheck 'system'. I just thought it would be a good idea to not accidentally delete some permissions if used everywhere.

shailesh91 commented 9 years ago

Hmm. Thanks for the assist.