Groups / Teams

[jlndk]

Hi.

Ealier today I created an issue on the original entrust repo, regarding the possibility of adding groups to the package.

However i just noticed the concerns regarding the maintainer of the original repo, so I looked up forks and stumbled upon this. Even though I'm gonna make the pull request myself, I would very much prefer the code going into a maintained codebase.

(BTW I've done a lot of explanation of my problem on the original issue, so please check it out if you're unsure of what I mean by groups)

Personally I think many people would benifit for this feature, since It's very common for application to be based around groups (like slack).

[santigarcor]

@jlndk i think it would be useful so if you can make a PR it would be great.

[jacurtis]

Yes, I think this would be a great addition and would be used by many people.

[erikwestlund]

I think this would be awesome.

[santigarcor]

I'm going to start working in that during week.

[jlndk]

My team found another way around this problem, but I'm glad someone else has started working on the feature :D

[santigarcor]

@jlndk , @erikwestlund , @jacurtis I've done most of the groups implementation. Could you please try it in this branch. In packagist it's the dev-groups-implementation one.

These are the things you need to have in mind:

• The detachRoles() when no param given, it'll only remove the roles that doesn't belongs to any group.
• The attachRole, hasRole and can work like @jlndk described here.
• The ability doesn't support groups yet.

I hope you can try that out, and if you find something please tell me so i can change it.

Cheers!

[santigarcor]

I just tagged a RC for the version with groups