-
Notifications
You must be signed in to change notification settings - Fork 9
Description
Expected behavior
The organization resource type should have a tenantType attribute, even in the beta SDK, as documented in beta API https://learn.microsoft.com/en-us/graph/api/resources/organization?view=graph-rest-beta. I also confirm that the beta API returns this field.
Actual behavior
The organization resource type doesn't have a tenantType attribute.
Note that it has a partnerTenantType:
msgraph-beta-sdk-java/src/main/java/com/microsoft/graph/models/Organization.java
Lines 158 to 165 in 03488c8
| /** | |
| * The Partner Tenant Type. | |
| * | |
| */ | |
| @SerializedName(value = "partnerTenantType", alternate = {"PartnerTenantType"}) | |
| @Expose | |
| @Nullable | |
| public PartnerTenantType partnerTenantType; |
But it doesn't seem to be the same thing (
tenantType should be a string "AAD" or "AAD B2C", whereas partnerTenantType is a whole object).
I noticed that this Java SDK in v1.0 has the tenantType:
https://github.com/microsoftgraph/msgraph-sdk-java/blob/c6825773ea32c1265427cf7843954356cd491074/src/main/java/com/microsoft/graph/models/Organization.java#L227-L234
It's documented in v1.0 too https://learn.microsoft.com/en-us/graph/api/resources/organization?view=graph-rest-1.0 so it isn't even a change between v1.0 and beta.
So it looks like a regression from the v1.0 to beta SDK.
Steps to reproduce the behavior
- open https://github.com/microsoftgraph/msgraph-beta-sdk-java/blob/dev/src/main/java/com/microsoft/graph/models/Organization.java
- look for
tenantType - notice it's missing