Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

importer-msgraph-metadata: Add workarounds for several bugs in Graph Api Spec #4519

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

philband
Copy link

This PR fixes some bugs I encountered whilst using the Graph SDK generated by pandora.

Model microsoft.graph.connectedOrganization

The field identitySources cannot be read-only, as it needs to be set when creating a new connected organization.
Docs Ref: https://learn.microsoft.com/en-us/graph/api/entitlementmanagement-post-connectedorganizations?view=graph-rest-beta&tabs=http#request

Model microsoft.graph.crossTenantAccessPolicyConfigurationPartner

The field tenantId cannot be read-only, otherwise it is impossible to create a new cross tenant access policy partner organization.
Docs Ref: https://learn.microsoft.com/en-us/graph/api/crosstenantaccesspolicy-post-partners?view=graph-rest-1.0&tabs=http#request

Model microsoft.graph.unifiedRoleManagementPolicy

The fields lastModifiedBy and lastModifiedDateTime must be read only, as they can only be modified by the backend service. Attempting to set them results in HTTP Error 400 Bad Request.
Docs Ref: https://learn.microsoft.com/en-us/graph/api/unifiedrolemanagementpolicy-update?view=graph-rest-1.0&tabs=http#request

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.

1 participant