Filter out deleted accounts

[fivetran-avinash]

PR Overview

This PR will address the following Issue/Feature: [#55]

This PR will result in the following new package version: 11.0

As deleted records are being filtered out here and in dbt_quickbooks, this changes the reporting and warrants a breaking change update.

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

Breaking Changes

• We are currently bringing in records that have previously been deleted, which could be impacting reporting accuracy. Thus, we introduced the _fivetran_deleted field to filter out these records from the following staging models:
  • stg_quickbooks__account
  • stg_quickbooks__bundle
  • stg_quickbooks__customer
  • stg_quickbooks__department
  • stg_quickbooks__item
  • stg_quickbooks__vendor
• As this will filter out deleted records that were previously being counted and we are adding a new field to these staging models, this will be a breaking change here and in dbt_quickbooks.

Documentation Updates

• Added the _fivetran_deleted field to src_quickbooks and stg_quickbook yml documentation.
• Added the _fivetran_deleted field to the above corresponding seed files in integration tests.

Contibutors

• @brandonrf94 #56

PR Checklist

Basic Validation

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

• dbt run –full-refresh && dbt test
• [NA] 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:

Models ran and passed, and the new filters did take out deleted records.

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

🧢

[fivetran-catfritz]

@fivetran-avinash This generally looks good. Just need some additional documentation update and then a docs regen!

[fivetran-catfritz]

This definition needs to be added to the other staging models:

• stg_quickbooks__bundle
• stg_quickbooks__customer
• stg_quickbooks__department
• stg_quickbooks__item
• stg_quickbooks__vendor

[fivetran-avinash]

Updated

[fivetran-avinash]

@fivetran-catfritz This is ready for re-review!

[fivetran-catfritz]

@fivetran-avinash updates look good. Approved!

[fivetran-joemarkiewicz]

LGTM