Add functionality for tierLicenseCapacity on sites#778
Add functionality for tierLicenseCapacity on sites#778jorwoods wants to merge 11 commits intotableau:developmentfrom
Conversation
6420e07 to
44ef0ed
Compare
44ef0ed to
f124b9f
Compare
|
#777 covers tierLicenseCapacity support, though it doesn't include the property_is_int annotation and user-quota limitation. Do you mind waiting until that one is checked in? Also I think the integer annotation on these are a bit tricky - I'm not sure if there is a limit to the value and -1 is a valid value to indicate reset |
|
This can wait until #777 is in. I don't like having the hardcoded upper bound in there either. I can replace it with a |
|
That works! I would keep the lower limit at 0 and pass in -1 as the |
|
@shinchris Question regarding the -1 to indicate reset: I had tested on my own server by passing If you think it should be able to accept None, I can either:
|
56c8ceb to
f41a717
Compare
|
@jorwoods I think it should be -1 instead. Passing in For the |
…rwoods/server-client-python into jorwoods/site_tiered_capacity
…rwoods/server-client-python into jorwoods/site_tiered_capacity
| error = 'You cannot set admin_mode to ContentOnly and also set a user quota' | ||
| raise ValueError(error) | ||
|
|
||
| reset = (None, -1,) |
There was a problem hiding this comment.
Why is this variable named "reset"? It could have always been empty, right?
This adds support to the SiteItems to allow it to be used to manage the license level specific counts.