Skip to content
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

[Enhancement] Added ability to add individual Permissions to User #876

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

vinayakkulkarni
Copy link

Basically #800 but with latest base fork (supporting Laravel 5.5)

Why?
Let's say I have a user, Joao, he is a reviewer who has x number of permissions defined in a role. But let's say I want Joao to edit other people's reviews too, but to do that, I'd need to add a new Role with additional Permission of editing other people's reviews too which would become chaotic as for each and every different type of permission, I'd need to create a new role.

Take a look at the screens:
Role:
role - reviewer

Permission:
permission - edit other reviews too

This highly increases the functionality of this awesome plugin and I bet the community would appreciate if they had an option to assign individual roles to user instead of creating temp roles for each and every permutation and combination of permissions and roles.

PS. I (might) have to update the README with more detailed example and usage.

PPS. I've not taken a look at the Tests yet, just copy-pasted them from @csi0n PR

@vinayakkulkarni
Copy link
Author

PHP 5.x tests failing cause:

Problem 1
    - Installation request for doctrine/inflector 1.2.x-dev -> satisfiable by doctrine/inflector[1.2.x-dev].
    - doctrine/inflector 1.2.x-dev requires php ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 2
    - Installation request for illuminate/cache 5.5.x-dev -> satisfiable by illuminate/cache[5.5.x-dev].
    - illuminate/cache 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 3
    - Installation request for illuminate/console 5.5.x-dev -> satisfiable by illuminate/console[5.5.x-dev].
    - illuminate/console 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 4
    - Installation request for illuminate/contracts 5.5.x-dev -> satisfiable by illuminate/contracts[5.5.x-dev].
    - illuminate/contracts 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 5
    - Installation request for illuminate/support 5.5.x-dev -> satisfiable by illuminate/support[5.5.x-dev].
    - illuminate/support 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 6
    - Installation request for illuminate/container 5.5.x-dev -> satisfiable by illuminate/container[5.5.x-dev].
    - illuminate/container 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 7
    - Installation request for illuminate/database 5.5.x-dev -> satisfiable by illuminate/database[5.5.x-dev].
    - illuminate/database 5.5.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 8
    - Installation request for mockery/mockery 1.0.x-dev -> satisfiable by mockery/mockery[1.0.x-dev].
    - mockery/mockery 1.0.x-dev requires php >=5.6.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 9
    - Installation request for mockery/mockery dev-master -> satisfiable by mockery/mockery[dev-master].
    - mockery/mockery dev-master requires php >=5.6.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 10
    - Installation request for sami/sami 4.0.x-dev -> satisfiable by sami/sami[4.0.x-dev].
    - sami/sami 4.0.x-dev requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 11
    - Installation request for sami/sami dev-master -> satisfiable by sami/sami[dev-master].
    - sami/sami dev-master requires php >=7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
  Problem 12
    - Installation request for twig/twig 2.x-dev -> satisfiable by twig/twig[2.x-dev].
    - twig/twig 2.x-dev requires php ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.

Everything passed in PHP 7

@blwsh
Copy link

blwsh commented Jul 15, 2019

Why the hell hasn't this been merged?

@vinayakkulkarni
Copy link
Author

No idea @uniben

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants