Skip to content

Conversation

nanjj
Copy link

@nanjj nanjj commented Mar 28, 2018

The error found when locationGroupId is None.

@coveralls
Copy link

coveralls commented Mar 28, 2018

Coverage Status

Coverage remained the same at 86.978% when pulling 65827f1 on nanjj:fix into 4fc25c8 on softlayer:master.

The error found when `locationGroupId` is None.
@allmightyspiff
Copy link
Member

locationGroupId needs a check to be None (or possibly '' on some occasions, I need to do some more testing to figure that one out), and not just the first result.

Thanks for the pull request, but I'm going to fix this in #956, hopefully by March 29.

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