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

Use ODataProperty directly to write Edm.Untyped property #923

Closed
xuzhg opened this issue May 16, 2023 · 2 comments · Fixed by #924
Closed

Use ODataProperty directly to write Edm.Untyped property #923

xuzhg opened this issue May 16, 2023 · 2 comments · Fixed by #924
Assignees
Labels
bug Something isn't working

Comments

@xuzhg
Copy link
Member

xuzhg commented May 16, 2023

Assemblies affected
Which assemblies and versions are known to be affected e.g. ASP.NET Core OData 8.x

Describe the bug
A clear and concise description of what the bug is.

Reproduce steps
As these comments mentioned,
https://github.com/OData/AspNetCoreOData/blob/main/src/Microsoft.AspNetCore.OData/Formatter/Serialization/ODataResourceSerializer.cs#L1338-L1351

When updating to ODL 7.16.0, please fix it.

Data Model
Please share your Data model, for example, your C# class.

EDM (CSDL) Model
Please share your Edm model, for example, CSDL file.
You can send $metadata to get a CSDL XML content.

Request/Response
Please share your request Uri, head or the request body
Please share your response head, body.

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Please share your call stack or any error message
Add any other context about the problem here.

@xuzhg xuzhg added the bug Something isn't working label May 16, 2023
@xuzhg xuzhg self-assigned this May 16, 2023
@corranrogue9
Copy link
Contributor

There is an ODL fix PR here

@corranrogue9
Copy link
Contributor

Once the new ODL version has been released, we should take that version and remove the code that @xuzhg has referenced in this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants