-
Notifications
You must be signed in to change notification settings - Fork 189
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
Graph api rest #2869
Graph api rest #2869
Conversation
54f81cd
to
febc157
Compare
💥 Acceptance test localApiTests-apiSpaces-ocis failed. Further test are cancelled... |
febc157
to
6e73ee7
Compare
dcd2c92
to
23458ba
Compare
Kudos, SonarCloud Quality Gate passed! |
@micbar why did you rip out godata? |
IIRC we're not using odata in favor of "traditional" REST. I initially implemented the drives endpoint using OData queries but I think it was too confusing (?) and it was moved to the usual verbs... which I don't like, I'd rather stick to the usual OData on msgraph, since we can leverage the power of OData 🤷♀️ There is more info in the PR that gets rids of odata on our graph |
Description
PATCH /drives/{drive-id}
Related Issue
Screenshots (if appropriate):
Types of changes
Checklist: