Cannot roll user out and make them a free account

[g1t-out]

• I have checked that the SDK documentation doesn't solve my issue.
• I have checked that the API documentation doesn't solve my issue.
• I have searched the Box Developer Forums and my issue isn't already reported (or if it has been reported, I have attached a link to it, for reference).
• I have searched Issues in this repo and my issue isn't already reported.

Description of the Issue

I'm unable to roll users out of my enterprise using Box's Python SDK. The call to update_user_by_id indicates it was successful, but the user remains in the enterprise.

Steps to Reproduce

self.box_client.users.update_user_by_id(user.id, enterprise=None)

If I run a report on activity in the enterprise, I see indications that this app updated the user, but nothing changed. This is the exact same things I saw when I reported this bug in the .Net SDK ( see this ).

The documentation says "Set this to null to roll the user out of the enterprise and make them a free user".

I think the issue here is that because null is the default for this attribute, you can't determine a value was actually passed. Because it is null, the enterprise attribute is removed from the final request and never sent to the Box API.

Expected Behavior

The account is out of the enterprise and now a free account.

Versions Used

Python SDK: box-sdk-gen 1.12.0
Python: 3.11.4

[g1t-out]

I also tried this:

from box_sdk_gen import NullValue

self.box_client.users.update_user_by_id(user.id, enterprise=NullValue())

And I get a 400 from Box. Oddly though, it looks like the request was correct, so I'm at a bit of a loss as to why it didn't work. I can see the request was sent to https://api.box.com/2.0/users/1234567890 with a body of {"enterprise": null}, but I get this from Box:

{
    "type": "error",
    "status": 400,
    "code": "bad_request",
    "context_info": {
        "errors": [
            {
                "reason": "invalid_parameter",
                "name": "user",
                "message":"Invalid value 'Box_User 1234567890'."
            }
        ]
    },
    "help_url": "http://developers.box.com/docs/#errors",
    "message": "Bad Request",
    "request_id": "nolqsvhysvbfwofz"
}

[g1t-out]

After a support ticket, I found the above error was because the user I was testing with also had a legal hold on them. After removing it, using the NullValue() does seem to work correctly. Perhaps using NullValue() instead of None needs to be documented?

[congminh1254]

Hi @g1t-out

Thanks for your feedback, I will discuss this topic with team soon and document should be update in the future.