Skip to content

Conversation

@VicDeo
Copy link
Member

@VicDeo VicDeo commented Jan 19, 2017

Description

Allow quota to be 'none'

Related Issue

Fixes #26975

Motivation and Context

If user has a quota set, it can't be reverted to Unlimited value

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@VicDeo VicDeo added this to the 10.0 milestone Jan 19, 2017
@VicDeo
Copy link
Member Author

VicDeo commented Jan 19, 2017

@PVince81 @jturcotte

@PVince81
Copy link
Contributor

Same issue for "Default" apparently, I'm getting "invalid value 'default'".

@VicDeo
Copy link
Member Author

VicDeo commented Jan 19, 2017

@PVince81 oops, missed that.
Should be fixed now

@PVince81
Copy link
Contributor

Works now, thanks 👍

@PVince81 PVince81 merged commit 578e390 into master Jan 19, 2017
@PVince81 PVince81 deleted the fix-26975 branch January 19, 2017 21:41
@individual-it individual-it mentioned this pull request Jan 31, 2017
9 tasks
@individual-it individual-it mentioned this pull request Feb 21, 2017
9 tasks
individual-it added a commit that referenced this pull request Mar 6, 2017
This sets up UI tests. It uses behat/mink/selenium to run the tests on
saucelabs.com
Included are some basic tests
1. Login as admin
2. Tests to cover the fix made in PR #26979
@individual-it individual-it mentioned this pull request Mar 6, 2017
9 tasks
DeepDiver1975 pushed a commit that referenced this pull request Mar 7, 2017
This sets up UI tests. It uses behat/mink/selenium to run the tests on
saucelabs.com
Included are some basic tests
1. Login as admin
2. Tests to cover the fix made in PR #26979
DeepDiver1975 pushed a commit that referenced this pull request Mar 7, 2017
This sets up UI tests. It uses behat/mink/selenium to run the tests on
saucelabs.com
Included are some basic tests
1. Login as admin
2. Tests to cover the fix made in PR #26979
DeepDiver1975 pushed a commit that referenced this pull request Mar 7, 2017
This sets up UI tests. It uses behat/mink/selenium to run the tests on
saucelabs.com
Included are some basic tests
1. Login as admin
2. Tests to cover the fix made in PR #26979
@lock
Copy link

lock bot commented Aug 4, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't set a user's quota back to Unlimited

3 participants