FEATURE - Add support for the detail_type field for Inventory Lines

[fivetran-joemarkiewicz]

Are you a Fivetran customer?

Fivetran identified feature request

Is your feature request related to a problem? Please describe.

The package currently has some involved where clauses within the int_quickbooks__invoice_double_entry model to filter out invoice subtotals and shipping costs.

dbt_quickbooks/models/double_entry_transactions/int_quickbooks__invoice_double_entry.sql

Line 116 in ce31cbb

where coalesce(invoice_lines.account_id, invoice_lines.sales_item_account_id, invoice_lines.sales_item_item_id, invoice_lines.item_id) is not null

While this logic does work, the connector now syncs the detail_type field that highlights if the field is a subtotal. The package would benefit from just filtering on this field opposed to a long coalesce argument to check for subtotals.

Describe the solution you'd like

The int_quickbooks__invoice_double_entry leverages the new detail_type field to filter out invoice line subtotals.

Describe alternatives you've considered

The current solution is the alternative which would be preferred to remove in favor of referencing the detail_type field.

Additional context

The detail_type field was recently added and would need to require a historical resync for users if they would like the new logic to work properly. This would mean the proposed feature request would result in a breaking change.

[fivetran-joemarkiewicz]

Closing this issue as it is now leveraged in the latest release via the above PR