Skip to content

Add/remove user permissions does not work. #1719

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.

Sign up for GitHub

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

Jump to bottom
Merged
samirdas merged 1 commit into from
Apr 19, 2016
Merged

Add/remove user permissions does not work. #1719

samirdas merged 1 commit into from
Apr 19, 2016

Conversation

@nicolasbrossard
Copy link
Contributor

@nicolasbrossard nicolasbrossard commented Apr 18, 2016

When there is only one user permission remaining, you cannot delete it. There might also be other cases where addition or removal of user permission fails. This use case should be tested thoroughly.

@nicolasbrossard nicolasbrossard added the Category: Bug PR or issue that aims to report or fix a bug label Apr 18, 2016
@nicolasbrossard nicolasbrossard added this to the 16.0 milestone Apr 18, 2016
@codecov-io
Copy link

codecov-io commented Apr 18, 2016

Current coverage is 15.19%

Merging #1719 into 16.04-dev will not affect coverage as of 61224f3

@@            16.04-dev   #1719   diff @@
=========================================
  Files             122     122       
  Stmts           19805   19807     +2
  Branches            0       0       
  Methods          1075    1075       
=========================================
  Hit              3010    3010       
  Partial             0       0       
- Missed          16795   16797     +2

Review entire Coverage Diff as of 61224f3

Powered by Codecov. Updated on successful CI builds.

@samirdas samirdas merged commit 5a54b2b into aces:16.04-dev Apr 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@xlecours xlecours

Labels

Category: Bug PR or issue that aims to report or fix a bug

Projects

None yet

Milestone

16.0

Development

Successfully merging this pull request may close these issues.

5 participants

@nicolasbrossard @codecov-io @samirdas @jstirling91 @xlecours