bugfix/double-entry-account-joins

fivetran · fivetran-joemarkiewicz · Nov 29, 2023 · Nov 21, 2023 · Nov 21, 2023 · Nov 28, 2023
commit b10353f76d143d230d264dd11fa96daf4c492966
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,14 @@
+# dbt_quickbooks v0.12.1
+[PR #] includes the following updates:
+
+## Bug Fixes
+- Adjusted the joins within the below intermediate double entry models to be `left join` as opposed to an `inner join`. This update was necessary as there was the possibility of the respective account cte joins to return no records if this was the case, the logic could erroneously remove transactions from the record.
+  - [int_quickbooks__bill_payment_double_entry](https://github.com/fivetran/dbt_quickbooks/blob/main/models/double_entry_transactions/int_quickbooks__bill_payment_double_entry.sql)
+  - [int_quickbooks__credit_memo_double_entry](https://github.com/fivetran/dbt_quickbooks/blob/main/models/double_entry_transactions/int_quickbooks__credit_memo_double_entry.sql)
+  - [int_quickbooks__deposit_double_entry](https://github.com/fivetran/dbt_quickbooks/blob/main/models/double_entry_transactions/int_quickbooks__deposit_double_entry.sql)
+  - [int_quickbooks__invoice_double_entry](https://github.com/fivetran/dbt_quickbooks/blob/main/models/double_entry_transactions/int_quickbooks__invoice_double_entry.sql)
+  - [int_quickbooks__payment_double_entry](https://github.com/fivetran/dbt_quickbooks/blob/main/models/double_entry_transactions/int_quickbooks__payment_double_entry.sql)
+
 # dbt_quickbooks v0.12.0
 [PR #103](https://github.com/fivetran/dbt_quickbooks/pull/103/files) includes the following updates:
 ## 🚘 Under the Hood

diff --git a/dbt_project.yml b/dbt_project.yml
@@ -1,7 +1,7 @@
 config-version: 2
 name: 'quickbooks'
 
-version: '0.12.0'
+version: '0.12.1'
 
 require-dbt-version: [">=1.3.0", "<2.0.0"]
 

diff --git a/docs/catalog.json b/docs/catalog.json
diff --git a/docs/index.html b/docs/index.html
diff --git a/docs/manifest.json b/docs/manifest.json
diff --git a/docs/run_results.json b/docs/run_results.json
diff --git a/integration_tests/.gitignore b/integration_tests/.gitignore
@@ -2,4 +2,5 @@ target/
 dbt_modules/
 dbt_packages/
 logs/
-env/
+env/
+package-lock.yml
diff --git a/integration_tests/dbt_project.yml b/integration_tests/dbt_project.yml
@@ -1,6 +1,6 @@
 name: 'quickbooks_integration_tests'
 
-version: '0.12.0'
+version: '0.12.1'
 
 profile: 'integration_tests'
 config-version: 2

diff --git a/models/double_entry_transactions/int_quickbooks__bill_payment_double_entry.sql b/models/double_entry_transactions/int_quickbooks__bill_payment_double_entry.sql
@@ -50,8 +50,8 @@ bill_payment_join as (
         bill_payments.department_id
     from bill_payments
 
-    inner join ap_accounts
-    on ap_accounts.source_relation = bill_payments.source_relation
+    left join ap_accounts
+        on ap_accounts.source_relation = bill_payments.source_relation
 ),
 
 final as (

diff --git a/models/double_entry_transactions/int_quickbooks__credit_memo_double_entry.sql b/models/double_entry_transactions/int_quickbooks__credit_memo_double_entry.sql
@@ -101,8 +101,8 @@ final as (
         'credit_memo' as transaction_source
     from credit_memo_join
 
-    inner join df_accounts
-    on df_accounts.source_relation = credit_memo_join.source_relation
+    left join df_accounts
+        on df_accounts.source_relation = credit_memo_join.source_relation
 )
 
 select *

diff --git a/models/double_entry_transactions/int_quickbooks__deposit_double_entry.sql b/models/double_entry_transactions/int_quickbooks__deposit_double_entry.sql
@@ -56,8 +56,8 @@ deposit_join as (
         on deposits.deposit_id = deposit_lines.deposit_id
         and deposits.source_relation = deposit_lines.source_relation
 
-    inner join uf_accounts
-    on uf_accounts.source_relation = deposits.source_relation
+    left join uf_accounts
+        on uf_accounts.source_relation = deposits.source_relation
 
 ),
 

diff --git a/models/double_entry_transactions/int_quickbooks__invoice_double_entry.sql b/models/double_entry_transactions/int_quickbooks__invoice_double_entry.sql
@@ -192,8 +192,8 @@ final as (
         end as transaction_source
     from invoice_filter
 
-    inner join ar_accounts
-    on ar_accounts.source_relation = invoice_filter.source_relation
+    left join ar_accounts
+        on ar_accounts.source_relation = invoice_filter.source_relation
 )
 
 select *

diff --git a/models/double_entry_transactions/int_quickbooks__payment_double_entry.sql b/models/double_entry_transactions/int_quickbooks__payment_double_entry.sql
@@ -84,8 +84,8 @@ final as (
         'payment' as transaction_source
     from payment_join
 
-    inner join ar_accounts
-    on ar_accounts.source_relation = payment_join.source_relation
+    left join ar_accounts
+        on ar_accounts.source_relation = payment_join.source_relation
 )
 
 select *