Salesforce custom object external
This modification:
- Updates Salesforce base API from version v53 to v62;
- Adds a new mapping to upsert a custom object by its Id.
Testing
- [X] Added unit tests for new functionality
- [ ] Tested end-to-end using the local server
- [ ] [If destination is already live] Tested for backward compatibility of destination. Note: New required fields are a breaking change.
- [ ] [Segmenters] Tested in the staging environment
- [ ] [Segmenters] [If applicable for this change] Tested for regression with Hadron.
hi @brennan - This is a Salesforce PR raised by an SA, so passing this PR on to you so it can be triaged.
New required fields detected
[!WARNING] Your PR adds new required fields to an existing destination. Adding new required settings/mappings for a destination already in production requires updating existing customer destination configuration. Ignore this warning if this PR is for a new destination with no active customers in production.
The following required fields were added in this PR:
- Destination: Salesforce (Actions), Action Field(s):operation,customObjectName,externalIdField,externalIdValue
Add these new fields as optional instead and assume default values in perform or performBatch block.
Codecov Report
:x: Patch coverage is 80.00000% with 5 lines in your changes missing coverage. Please review.
:warning: Please upload report for BASE (main@078cbaa). Learn more about missing BASE report.
:warning: Report is 316 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #2485 +/- ##
=======================================
Coverage ? 78.50%
=======================================
Files ? 1037
Lines ? 18812
Branches ? 3576
=======================================
Hits ? 14769
Misses ? 2838
Partials ? 1205
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
hi @seg-leonelsanches just doing some tidy up - can we close this PR?