Skip to content

Feature: Add transaction_accounting_lines_pass_through_columns support #207

Closed

Description

@anorth848

Summary

The package supports pass-through columns for transactions, transaction_lines, accounts, customers, departments, locations, subsidiaries, vendors, items, classes, consolidated_exchange_rates, nexuses, entities, and accounting_periods — but not for transaction_accounting_lines.

Current behavior

stg_netsuite2__transaction_accounting_lines_tmp pulls all columns from the raw source (including paymentamountunused and paymentamountused), but stg_netsuite2__transaction_accounting_lines uses an explicit column list in the final CTE that drops them. There is no transaction_accounting_lines_pass_through_columns variable, no fivetran_utils.add_pass_through_columns call in get_netsuite2_transactionaccountingline_columns(), and no fill_pass_through_columns call in the staging model.

Requested behavior

Add transaction_accounting_lines_pass_through_columns support matching the existing pattern used by transaction_lines:

  1. Add transaction_accounting_lines_pass_through_columns: [] default var in dbt_project.yml
  2. Add {{ fivetran_utils.add_pass_through_columns(columns, var('transaction_accounting_lines_pass_through_columns')) }} to get_netsuite2_transactionaccountingline_columns()
  3. Add {{ netsuite.fill_pass_through_columns(var('transaction_accounting_lines_pass_through_columns', [])) }} to the final select in stg_netsuite2__transaction_accounting_lines.sql

Use case

Payment reporting requires paymentamountunused and paymentamountused per accounting book (for multibook environments). Without pass-through support, consumers must bypass the package staging layer and read the raw source directly, breaking the layered architecture the package provides.

Package version

v1.6.0

Activity

  1. fivetran-jamie commented on Aug 5, 2026

    @fivetran-jamie
    Contributor

    Hi @anorth848 thanks for opening this, definitely makes sense to add!

    I'll slate this in for our next sprint (starts in two weeks). While we're at it, are there are any tables you'd want passthrough columns for?

  2. anorth848 commented on Aug 10, 2026

    @anorth848
    Author

    Hey @fivetran-jamie , thanks for your response and for prioritizing this! This is all we need as far as I can tell/anticipate. Thank you again!

  3. linked a pull request that will close this issueRelease/1.8.0 #210on Aug 26, 2026
  4. fivetran-jamie commented on Aug 31, 2026

    @fivetran-jamie
    Contributor

    @anorth848 This is now available in v1.8.0 via the transaction_accounting_lines_pass_through_columns variable!

  5. anorth848 commented on Aug 31, 2026

    @anorth848
    Author

    @anorth848 This is now available in v1.8.0 via the transaction_accounting_lines_pass_through_columns variable!

    Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions