Cast deposit_to_account_id as integer

fivetran / dbt_quickbooks_source

Data models for QuickBooks built using dbt.

https://fivetran.github.io/dbt_quickbooks_source/

Apache License 2.0

5 stars 18 forks source link

Cast deposit_to_account_id as integer #8

Closed jamesrayoub closed 3 years ago

jamesrayoub commented 3 years ago

In the Invoice staging model, deposit_to_account_id is currently represented as a string, but since we cast account_id in the Accounts model as an integer, this FK should also be cast as an int

fivetran-joemarkiewicz commented 3 years ago

Thanks for catching this @jamesrayoub! We are planning an enhancement to the dbt_quickbooks package to include a monthly P&L model. We will incorporate this fix in those updates.

fivetran-joemarkiewicz commented 3 years ago

This issue has been addressed within PR #9 and is now live within Release v0.2.0.

Keep an eye out on the dbt hub for when it is officially released!