New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syncs contacts based on `versionTimestamp` #58

Merged
merged 1 commit into from Jan 3, 2018

Conversation

Projects
None yet
4 participants
@jaeger401
Contributor

jaeger401 commented Dec 18, 2017

This pull request addresses a long-standing issue with list memberships in which memberships are dependent on whether or not a contact has been updated, and thus new dynamic list memberships are frequently not captured.

HubSpot has recently exposed a new pair of fields (version & versionTimestamp) which are returned when the includeVersion query parameter is set to true when calling the contacts APIs. These new fields track the internal versions of the contacts, which includes updates due to new dynamic list memberships. Updates to other contact properties will also trigger a new versionTimestamp, so no prior functionality is lost. Thanks to @jaredmiwilliams for the change.

I've tested a number of scenarios against our dev HubSpot account to confirm functionality, and everything works as expected.

Syncs contacts based on `versionTimestamp`
This commit addresses a long-standing issue with list memberships in
which memberships are dependent on whether or not a contact has been
updated, and thus new dynamic list memberships are frequently not
captured.

HubSpot has recently exposed a new pair of fields (`version` &
`versionTimestamp`) which are returned when the `includeVersion` query
parameter is set to `true` when calling the `contacts` APIs. These new
fields track the internal versions of the contacts, which includes
updates due to new dynamic list memberships. Updates to other contact
properties will also trigger a new `versionTimestamp`, so no prior
functionality is lost.
@jdwyah

This comment has been minimized.

Contributor

jdwyah commented Dec 18, 2017

@jaredmiwilliams #fyi & thankyou

@nick-mccoy

This comment has been minimized.

Contributor

nick-mccoy commented Dec 19, 2017

Thanks for the contribution @jaeger401! I'm going to run a few tests to validate your changes and then I'll update with results.

erinkcochran87 added a commit to stitchdata/docs that referenced this pull request Dec 29, 2017

@psantacl psantacl merged commit c88f0b6 into singer-io:master Jan 3, 2018

@erinkcochran87 erinkcochran87 referenced this pull request May 15, 2018

Merged

Hubspot: v2.0 #7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment