-
Notifications
You must be signed in to change notification settings - Fork 191
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
fix unmount item from share #8827
Conversation
88d7b7b
to
45ec0b8
Compare
5f585b7
to
450a3fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some wording suggestions. Looks fine otherwise.
WEB_COMMITID=5367936f30b07c0832fb8af09f9f7719cccf7038 | ||
WEB_BRANCH=master | ||
WEB_COMMITID=9b9616b8429432002d06c4a2c8f252cbe08fb735 | ||
WEB_BRANCH=changeResponseCodeDelete |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this really be part of the PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. This seems to be needed to resolve a circular dependency with web. Needs to be revert once the related web changes are merged.
Bugfix: Fix unmount item from share | ||
|
||
We fixed the issue when unmount item from share will result in 200 status code | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We fixed the status code returned for the request to delete a driveitem.
|
fix unmount item from share
Description
We fixed the issue when unmount item from share will result in 200 status code
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: