Releases/v0.17.latest

[fivetran-jamie]

PR Overview

This PR will address the following Issue/Feature:

• [Feature] Add subsidiary currency and transaction amount #150 by way of merging Added transaction_amount to balance_sheet & income_statement #151
• [Bug] Duplicate Balance Sheet and Transaction Sheet Records #128 and [Feature] Switch off unnecessary config logic when not materializing as incremental #140 by way of merging Remove incremental logic from int_netsuite2__tran_with_converted_amounts #153

This PR will result in the following new package version:

v0.17.0

Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:

This release involves breaking changes and will require running a full refresh.

Bug Fixes

• Adjusted the materialization of the int_netsuite2__tran_with_converted_amounts model from incremental to ephemeral. There was a previous issue with the incremental strategy that could potentially produce duplicate records in certain situations (PR #153).
  • Our team has determined that the most sensible path forward is to remove the incremental logic for this model entirely, as its performance benefits have proved marginal and an ephemeral materialization will avoid any duplication issues.

This is a Breaking Change, as int_netsuite2__tran_with_converted_amounts will no longer materialize in the warehouse.

Feature Updates

• Added two fields to the netsuite2__balance_sheet and netsuite2__income_statement models (PR #151):
  • transaction_amount
  • subsidiary_currency_symbol

This change will require running a full refresh, as we are adding new fields to incrementally materialized models.

PR Checklist

Basic Validation

Please acknowledge that you have successfully performed the following commands locally:

• dbt run –full-refresh && dbt test
• dbt run (if incremental models are present) && dbt test

Before marking this PR as "ready for review" the following have been applied:

• The appropriate issue has been linked, tagged, and properly assigned
• All necessary documentation and version upgrades have been applied
• docs were regenerated (unless this PR does not include any code or yml updates)
• BuildKite integration tests are passing
• Detailed validation steps have been provided below

Detailed Validation

Please share any and all of your validation steps:

full refresh validation tests pass

as do incremental

If you had to summarize this PR in an emoji, which would it be?

💱

[fivetran-catfritz]

@fivetran-jamie Left a couple suggested but otherwise approved!