Merge branch 'main' into more_generic

dbt-labs · Sep 9, 2021 · 8ebbc1e · 8ebbc1e
2 parents 8b69516 + b08b27e
commit 8ebbc1e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/models/customers.sql b/models/customers.sql
@@ -26,7 +26,7 @@ customer_orders as (
         count(order_id) as number_of_orders
     from orders
 
-    group by 1
+    group by customer_id
 
 ),
 
@@ -41,7 +41,7 @@ customer_payments as (
     left join orders on
          payments.order_id = orders.order_id
 
-    group by 1
+    group by orders.customer_id
 
 ),
 

diff --git a/models/orders.sql b/models/orders.sql
@@ -25,7 +25,7 @@ order_payments as (
 
     from payments
 
-    group by 1
+    group by order_id
 
 ),
 
@@ -47,6 +47,7 @@ final as (
 
     from orders
 
+
     left join order_payments
         on orders.order_id = order_payments.order_id
 

diff --git a/models/staging/stg_payments.sql b/models/staging/stg_payments.sql
@@ -15,7 +15,7 @@ renamed as (
         order_id,
         payment_method,
 
-        --`amount` is currently stored in cents, so we convert it to dollars
+        -- `amount` is currently stored in cents, so we convert it to dollars
         amount / 100 as amount
 
     from source