-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add Service Hub models #27
Comments
opened #28 to address this |
Hi @jamesrayoub PR #30 implements the Service Hub changes you initiated within PR #28. We are holding off merging the PR to master until the additional tables you requested with the Support Portal are added at the connector level. This way we may fully capture the Service Hub models. |
Great, thanks for the update @fivetran-joemarkiewicz |
Hi @jamesrayoub, I was able to add the Additionally, in response to your PR #35 I leveraged the code you provided and integrated it with how we have handled pass through columns in the past. The modifications I made should hopefully make declaring the variable a bit easier and will be consistent with our previous package pass through variables. The approach I took was to only provide the opinionated "standard" fields for the Let me know if you have any questions on this approach. Also, feel free to install this working branch as a dependency prior to us merging to master and releasing a new version as I would greatly appreciate your feedback and contributions beforehand!
|
Fivetran added the Service Hub tables to the connector last year and the following staging models should be created
ticket
ticket_company
ticket_contact
ticket_deal
ticket_engagement
ticket_property_history
The text was updated successfully, but these errors were encountered: