[Feature] Adding passthrough Custom Attributes also for Conversations about dbt_intercom

[fivetran-joemarkiewicz]

@fivetran-catfritz in addition to working through this feature request, would you be able to remove the test here as it has been shown to not be accurate for all implementations.

We were also able to identify that per Intercom's documentation, this test can in fact fail and the data is not incorrect. See here for the Intercom docs explaining the nobody_admin assignee_type. In these cases, the assignee_id will be null. As such, we will want to remove the test.

[ccbrandenburg]

Awesome ;)

+1 on the above test. It's failing for us too but the data is inherently correct.

[fivetran-catfritz]

@ccbrandenburg Thank you for submitting this feature request. We have added this to our current projects and plan to have a test branch for you to try in the next week or so. We'll keep you posted!

[fivetran-catfritz]

Hi @ccbrandenburg I have a created a test branch for you to try out. You'll just need to add the below to you packages.yml in place of the normal dbt_intercom.

- git: https://github.com/fivetran/dbt_intercom.git
  revision: feature/add-conversation-passthroughs
  warn-unpinned: false

The var to add your dbt_project.yml for your list of passthrough columns for conversation_history is intercom__conversation_history_pass_through_columns.

Please let me know how this works for you and if it is producing what you would expect!

[ccbrandenburg]

Unfortunately I can't get it to work.

Deleted the fivetran-stripe package to make sure there are no dependencies causing issues.

Added this to the dbt_project.yml:
intercom__conversation_history_pass_through_columns: [custom_customer, custom_description]
But the compiled sql fails and is not pulling in the two field names. I tried wrapping the fields with ' ' and " " but did not help.

[fivetran-catfritz]

Hi @ccbrandenburg thank you for trying this out! I took a closer look and made some adjustments that are now proving out on my end. Would you mind giving it another whirl?

[ccbrandenburg]

@fivetran-catfritz works like a charm! Tests are passing now too.

Thanks for the quick package improvement

[fivetran-catfritz]

@ccbrandenburg Awesome! Thanks again for testing that out. We will be pushing out a new release with these changes in the next day or so!

[fivetran-catfritz]

The update has been released. Closing out the issue!