Skip to content
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

source-hubspot-native: bindings for owners, deal pipelines, products, and line items #2164

Merged
merged 3 commits into from
Nov 22, 2024

Conversation

williamhbaker
Copy link
Member

@williamhbaker williamhbaker commented Nov 22, 2024

Description:

Adds new bindings for owners, deal pipelines, products, and line items.

As noted in the commit messages, for products and line items we don't have data in our test account and I'm not going to add any, so the test snapshots are unchanged for those. I did manually test that the results are as expected on a a HubSpot account that has data for those bindings though.

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

HubSpot native docs will need updated.

Notes for reviewers:

(anything that might help someone review this PR)


This change is Reviewable

Adds 2 new snapshot bindings, for owners and deal pipelines.

Also turned off the "properties" bindings for tests since it was a huge response
that was spamming everything else out.
Adds a new incremental binding for "products". This probably could be a snapshot
binding but it is easier to use the existing property fetching stuff we have for
other incremental bindings already, and it's theoretically possible that
somebody will have a large number of rapidly changing products.

We don't have any "product" objects in our test account and I don't feel like
figuring out how to add any, so the capture snapshot output is unchanged.
Adds a new incremental binding for "line items".

Like "products", we don't have any "line items" in our test account and I'm not
going to put any in there so the capture snapshot output is unchanged.
Copy link
Contributor

@Alex-Bair Alex-Bair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I just had a couple questions to solidify my understanding.

@williamhbaker williamhbaker merged commit e0cdc98 into main Nov 22, 2024
70 of 78 checks passed
@williamhbaker williamhbaker deleted the wb/hsn-more-streams branch November 22, 2024 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants