search

MeltanoLabs / tap-salesforce

Singer.io tap for the Salesforce API
GNU Affero General Public License v3.0
1 stars 30 forks source link

API version should be a configuration #12

Open MeltyBot opened 3 years ago

MeltyBot commented 3 years ago

Migrated from GitLab: https://gitlab.com/meltano/tap-salesforce/-/issues/13

Originally created by @bwise on 2021-11-04 16:04:51

API version is currently hard coded in the tap (https://gitlab.com/meltano/tap-salesforce/-/blob/master/tap_salesforce/salesforce/__init__.py#L212) This should be a configuration option.

Impact: I noticed I was missing fields in my replication of Salesforce records. This was fundamentally caused by the api version being too low for the field I wanted to replicate.

MeltyBot commented 2 years ago

View 1 previous comment from the original issue on GitLab

s7clarke10 commented 9 months ago

I totally support this change, would be a fairly quick change rather than making changes like this : https://github.com/MeltanoLabs/tap-salesforce/pull/55