Merged PR 3379: Fixes null on dates

# Description

Fixes dbt test - I forgot to exclude dates when these are null.

# 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.
- [NA] I have checked for DRY opportunities with other models and docs.
- [NA] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Fixes null on dates

Related work items: #23453
This commit is contained in:
Oriol Roqué Paniagua 2024-10-30 13:33:11 +00:00
parent 8ddb695947
commit 22d5b75931
2 changed files with 2 additions and 0 deletions

View file

@ -20,4 +20,5 @@ left join
{{ ref("int_kpis__dimension_daily_accommodation") }} as icmas
on icuh.id_deal = icmas.id_deal
and icvr.verification_estimated_completed_date_utc = icmas.date
where icvr.verification_estimated_completed_date_utc is not null
group by 1, 2, 3, 4

View file

@ -20,4 +20,5 @@ left join
{{ ref("int_kpis__dimension_daily_accommodation") }} as icmas
on icuh.id_deal = icmas.id_deal
and icvr.verification_estimated_started_date_utc = icmas.date
where icvr.verification_estimated_started_date_utc is not null
group by 1, 2, 3, 4