Force.com-Toolkit-for-NET icon indicating copy to clipboard operation
Force.com-Toolkit-for-NET copied to clipboard

Should we extend the SuccessResponse Object?

Open Zhaph opened this issue 4 years ago • 0 comments
trafficstars

Since API v46, the response for the Upsert Request has changed in two significant ways

  1. The created parameter has been added to the successful response object.
  2. Updates now return a 200 OK with a full body response instead of an empty response.

Previously I'd been using the existence of an Id in the response to indicate that the Upsert had created an object, and the lack of one to indicate an update to an existing object - this is no longer true.

At the moment, my code doesn't do much apart from logging the difference, but this might be more of an impact for others.

  • Does anyone else see the need to expose the Created property?
  • Should this just be added to the standard SuccessResponse object, or should we look to modify the calls used by upsert from the general Patch requests to UpsertPatch requests that return an UpsertSuccessResponse?

Zhaph avatar Nov 03 '21 13:11 Zhaph