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

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

Closed
1 of 4 tasks
fivetran-joemarkiewicz opened this issue Jan 17, 2023 · 3 comments
Closed
1 of 4 tasks
Assignees
Labels
good first issue Good for newcomers priority:p3 Affects many users; can wait type:bug Something is broken or incorrect update_type:models Primary focus requires model updates

Comments

@fivetran-joemarkiewicz
Copy link
Contributor

fivetran-joemarkiewicz commented Jan 17, 2023

Is there an existing issue for this?

  • I have searched the existing issues

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:
- name: "vendors_custom_field"
alias: "vendors_field"
items_pass_through_columns:
- name: "items_custom_field"
alias: "items_field"

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?

  • Yes.
  • Yes, but I will need assistance and will schedule time during our office hours for guidance
  • No.
@fivetran-joemarkiewicz fivetran-joemarkiewicz added bug Something isn't working good first issue Good for newcomers priority:p3 Affects many users; can wait type:bug Something is broken or incorrect update_type:models Primary focus requires model updates and removed bug Something isn't working labels Jan 17, 2023
@fivetran-jamie
Copy link
Contributor

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
Copy link
Contributor Author

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
Copy link
Contributor Author

Closing as this has been addressed in the above PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers priority:p3 Affects many users; can wait type:bug Something is broken or incorrect update_type:models Primary focus requires model updates
Projects
None yet
Development

No branches or pull requests

2 participants