Skip to content

Feature/rbac user params merged#423

Merged
steinkel merged 9 commits into
developfrom
feature/rbac-user-params-merged
Sep 21, 2016
Merged

Feature/rbac user params merged#423
steinkel merged 9 commits into
developfrom
feature/rbac-user-params-merged

Conversation

@steinkel

Copy link
Copy Markdown
Member

No description provided.

@steinkel steinkel added this to the 3.3.0 milestone Sep 12, 2016
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.2%) to 80.561% when pulling 9658a20 on feature/rbac-user-params-merged into ae71a10 on develop.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.3%) to 80.599% when pulling 9473a0b on feature/rbac-user-params-merged into ae71a10 on develop.

@steinkel
steinkel force-pushed the feature/rbac-user-params-merged branch from 9473a0b to a402d37 Compare September 21, 2016 20:34
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.2%) to 80.599% when pulling a402d37 on feature/rbac-user-params-merged into a0dbd4a on develop.

@steinkel
steinkel merged commit 19f4879 into develop Sep 21, 2016
@steinkel

Copy link
Copy Markdown
Member Author

Thank you @Xymanek, nice improvements to our not-so-SimpleRbacAuthorize class

@steinkel
steinkel deleted the feature/rbac-user-params-merged branch September 21, 2016 20:48
@Xymanek

Xymanek commented Sep 22, 2016

Copy link
Copy Markdown
Contributor

Oh I thought this was merged already... Anyway glad I could help 😄

@Xymanek

Xymanek commented Sep 22, 2016

Copy link
Copy Markdown
Contributor

Also if

not-so-Simple

creates confusion I guess it can be renamed/split in a major version

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.

3 participants