fivetran / dbt_netsuite

Data models for Fivetran's Netsuite connector, built using dbt.
https://fivetran.github.io/dbt_netsuite/
Apache License 2.0
36 stars 34 forks source link

Add locations pass through columns in transaction_details #124

Closed tom-rb closed 1 month ago

tom-rb commented 2 months ago

Please provide your name and company Antonio Ruby Barreto, Fastned

Link the issue/feature request which this PR is meant to address https://github.com/fivetran/dbt_netsuite/issues/123

Detail what changes this PR introduces and how this addresses the issue/feature request linked above. This PR includes the locations_pass_through_columns in netsuite2__transaction_details table, besides the location_id column. This aligns with the already existing behavior of accounts_pass_through_columns and departments_pass_through_columns.

How did you validate the changes introduced within this PR? Manual testing in my company setup (changing the file directly in dbt_installed_packages/).

Which warehouse did you use to develop these changes? Databricks

Did you update the CHANGELOG?

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

Provide an emoji that best describes your current mood :sunglasses: