From 39fc6a7d1353c9ed38f0afb7d451b64faeb4abc8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Oriol=20Roqu=C3=A9=20Paniagua?= Date: Wed, 5 Mar 2025 14:57:57 +0000 Subject: [PATCH] Merged PR 4618: Bugfix - Ensure the PMS is active # Description Very silly mistake - affects many reports! @ let me know if there's other use cases. I believe that's all of the computations, but you have this more fresh than me. # Checklist - [X] The edited models and dependants run properly with production data. - [X] The edited models are sufficiently documented. - [X] The edited models contain PK tests, and I've ran and passed them. - [ ] I have checked for DRY opportunities with other models and docs. - [ ] I've picked the right materialization for the affected models. # Other - [ ] Check if a full-refresh is required after this PR is merged. Related work items: #28196 --- models/intermediate/core/int_core__deal.sql | 1 + models/intermediate/core/int_core__user_host.sql | 1 + 2 files changed, 2 insertions(+) diff --git a/models/intermediate/core/int_core__deal.sql b/models/intermediate/core/int_core__deal.sql index 667fd8d..08d5bcc 100644 --- a/models/intermediate/core/int_core__deal.sql +++ b/models/intermediate/core/int_core__deal.sql @@ -84,6 +84,7 @@ with stg_core__integration_type scit on sci.id_integration_type = scit.id_integration_type left join int_core__unified_user uu on sci.id_superhog_user = uu.id_user + where sci.is_active = true ), integrations_per_deal as ( diff --git a/models/intermediate/core/int_core__user_host.sql b/models/intermediate/core/int_core__user_host.sql index 36f22a7..ed57b73 100644 --- a/models/intermediate/core/int_core__user_host.sql +++ b/models/intermediate/core/int_core__user_host.sql @@ -40,6 +40,7 @@ with stg_core__integration_type scit on sci.id_integration_type = scit.id_integration_type left join int_core__unified_user uu on sci.id_superhog_user = uu.id_user + where sci.is_active = true ), integrations_per_host as ( select