fivetran / dbt_netsuite_source

Data models for Fivetran's Netsuite source package, built using dbt.
https://fivetran.github.io/dbt_netsuite_source/
Apache License 2.0
14 stars 20 forks source link

[Bug] Persist Item and Vendor Passthrough Columns for Netsuite2 #25

Closed fivetran-joemarkiewicz closed 1 year ago

fivetran-joemarkiewicz commented 1 year ago

Is there an existing issue for this?

Describe the issue

PR #23 added passthrough columns for items and vendors into the Netsuite1 models. However, the same changes were not applied to the Netsuite2 models in the package. The passthrough columns should work as intended for both the Netsuite and Netsuite2 forms of this package.

Relevant error log or model output

M/A

Expected behavior

Passthrough columns defined for items and vendors persist when using both Netsuite and Netsuite2 models.

dbt Project configurations

vars: vendors_pass_through_columns:

Package versions

v0.6.0

What database are you using dbt with?

postgres, redshift, snowflake, bigquery, databricks

dbt Version

v1.3.1

Additional Context

The code within the stg_netsuite__items and stg_netsuite__vendors can simply be copy/pasted into the Netsuite2 counterpart models to address this issue.

Are you willing to open a PR to help address this issue?

fivetran-jamie commented 1 year ago

i noticed that these passthrough columns are not in the transform package, though these tables are indeed brought into the transaction_details model.... should we also add the vendor/item passthrough columns to the transform package while we're at it? @fivetran-joemarkiewicz

fivetran-joemarkiewicz commented 1 year ago

Following our conversation offline, let's plan to not persist the passthrough columns through for now. However, if we could create a complimentary FR to track this idea that would be great!

fivetran-joemarkiewicz commented 1 year ago

Closing as this has been addressed in the above PRs.