Fixes #331 - Added ExternalIdFieldName to JobInfo, and overloads to po…

[Zhaph]

This fixes #331

Added ExternalIdFieldName to the JobInfo class along with auto filling specified property to hide from XML if not required.

Added overloads to the RunJobAsync, RunJobAsyncAndPollAsync and CreateJobAsync methods on ForceClient to allow ExternalIdFieldName to be supplied, along with an argument check when operation type is Upsert.

Added optional test in BulkForceClientFunctionalTests to verify going forward - requires a new External Id field to be added to an object (I've added Unique_Email__c to Contact because that's where I needed it most, but happy to move it).

[mschmidt-healthx]

How long does it typically take for a fix like this to be merged and released?

[wadewegner]

Thanks, @Zhaph.

@mschmidt-healthx I can try to prioritize it this week. I haven't looked at the details. Do you need it soon?

[mschmidt-healthx]

We're getting ready to start a new project and will need to use bulk upsert - so trying to plan out whether I need to allow more time to work around this. It doesn't seem like it would be too big of a deal to manually call the job create API and then walk through the rest of RunJobAsync with a crafted JobInfoResult but it'd be nice if I didn't have to ;)

Just a happy coincidence that I noticed that bulk upserts wouldn't work through the sdk this morning and a pull request had been submitted less than a day ago to fix it.

[Zhaph]

No worries @wadewegner apologies for the flurry of commits after the pull request. I've got an updated test using the Campaign object and native fields if that would help? Apparently you can use (almost) any indexed field as the External Id for an upsert but you do need to specify it.

[cappemic]

@Zhaph thanks for this! Ran into the same problem myself and your changes worked great.

[tkamesh08]

@wadewegner

I terribly need this bulk upsert functionality, can you please let me know when this will be updated in master and Nuget package.

Thanks

[wadewegner]

Another impressive contribution, @Zhaph! Thank you so much. I'm terribly sorry it's taken so long to merge.