Merged PR 4618: Bugfix - Ensure the PMS is active

# Description

Very silly mistake - affects many reports!

@<Joaquin Ossa> 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
This commit is contained in:
Oriol Roqué Paniagua 2025-03-05 14:57:57 +00:00
parent 5ddbe5d5af
commit 39fc6a7d13
2 changed files with 2 additions and 0 deletions

View file

@ -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 (

View file

@ -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