-
Notifications
You must be signed in to change notification settings - Fork 35
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
adds inner error description #110
Conversation
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
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.
This new functionality should be optional so that existing use of the transform do not need to be updated.
Co-authored-by: Michael Mainer <MIchaelMainer@users.noreply.github.com>
@zengin do you have any idea why the workflow is failing at this point? |
Co-authored-by: Mustafa Zengin <muzengin@microsoft.com>
@baywet now the error looks like on how the transform is applied. The rule removes the whole microsoft.graph namespace for some reason. |
Maybe the chose tag should wrap around the template one? (not in front of my computer right now) |
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
there, the issue was caused by two things:
|
inserts a description for the inner error type as prepration work for Kiota.
We're not describing the error itself as it's brought in the conversion process by OData conventions.
related microsoft/kiota#411
related microsoft/OpenAPI.NET.OData#167
More reading https://docs.microsoft.com/en-us/graph/errors
(I believe the inner error description is partly wrong, as it doesn't inherit from error according to the odata spec)