Implement Bulk Operations in WebAPI#2455
Conversation
* downgrade the newtonsoft.json version to 6.0.4 for classic * change the newtonsoft.json version in classic test
(OData#1560) * Don't assume port 80 when generating nextLink in ASP.NET Core OData#1559 If the port isn't set on `request.Host`, port 80 was assumed previously. If the app is running behind a reverse proxy, and the port was not forwarded, this can result in invalid URLs - for example an https URL using port 80 Instead, we just omit the port completely if it's not explictly set. * Add unit tests for GetNextPageLink * Add copyright notice
…#1588) - Null check on ODataQueryParameterBindingAttribute for Net Core
This is a regression from 5.x to 6.x/7.x. It looks like the original fix was never merged to the 6.0 development branch. Issue: OData#736
- Add comma before MyLevel property.
This is addresses the unhelpful error message described in this GitHub issue: OData#1421
Improving the error message for when an unknown property is deserialized
We can throw an exception if its null, which would change the current behaviour, but i guess thats fine right In reply to: 828778014 Refers to: src/Microsoft.AspNet.OData.Shared/Formatter/Serialization/ODataSerializerContext.cs:290 in ab656df. [](commit_id = ab656df, deletion_comment = False) |
- This was caught when moving ServiceCollection to another assembly (type forwarded).
* BulkInsert1 draft * Bulk insert changes * deleted old test * updates * Update ODataResourceSetWrapper.cs * updates * Updated addressing comments * updates * updates updates * Updates * Updates * Minor updates * comments * updates * update publicapi for core * Address comments * Cleanup and additional tests Cleanup and additional tests * Updated code * BulkInsert1 draft * Bulk insert changes * deleted old test * updates * updates * Updated addressing comments * updates * updates updates * Updates * Updates * Minor updates * updates * Address comments * Cleanup and additional tests Cleanup and additional tests * DataAnnotationException updates * comments * small updates * updates * small update * updates * Updates * Update DeltaSetOfT.cs * Updates with Patch * updates * updates * Update WebHostTestFixture.cs * updates * Update DeltaOfTStructuralType.cs * Updates * Updates for serializer etc * Update WebHostTestFixture.cs * updates * updates * updates * Bulk Operations Updates
f9f672d to
a7a67fc
Compare
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
4c43a84 to
ddfd3dd
Compare
Issues
Implements Bulk Operation in webAPI
Description
Briefly describe the changes of this pull request.
Checklist (Uncheck if it is not completed)
Additional work necessary
If documentation update is needed, please add "Docs Needed" label to the issue and provide details about the required document change in the issue.