-
Notifications
You must be signed in to change notification settings - Fork 4
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
ODataService.request: removing all @odata metadata fields was a wrong decision #60
Comments
It probably fixed the contradiction between github and npmjs sources, but not the issue with functionality itself. |
The npm package is not updated because the npm token is not defined under Github Actions Secrets. |
@AimTune That was not the point of this issue. The point was to remove back the code that cleared metadata and leave it as it was before |
First, if it is done, is should be done recursively, not only on single entity, but also on arrays.
And second, it should not be done at all, because e.g. @odata.etag is required for optimistic concurrency when RowVersion field is not included in $select clause or is not present in DTO model.
Presence of odata metadata in response should be controlled by "metadata" param of "accept" http header. It may be a param in ODataService constructor
The text was updated successfully, but these errors were encountered: