Merged PR 4105: Fixes exclusion of MRR after name change

# Description

After the name change of the metric, the exclusion in reporting for the ongoing month and previous month is not working for the new MRR metric. This PR aims to fix it.

# Checklist

- [ ] The edited models and dependants run properly with production data.
- [ ] The edited models are sufficiently documented.
- [ ] 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.

Fixes exclusion of MRR after name change

Related work items: #26621
This commit is contained in:
Oriol Roqué Paniagua 2025-01-19 09:37:07 +00:00
parent 3f3b99cdf6
commit 873402fd8e

View file

@ -68,7 +68,7 @@ where
or lower(metric) like '%resolutions%'
or lower(metric) like '%invoiced%'
or lower(metric) like '%retained%'
or lower(metric) like '%expected mrr%'
or lower(metric) like '%mrr%'
or lower(metric) like '%damage host%'
)
and {{ is_date_before_previous_month("date") }}
@ -80,7 +80,7 @@ where
or lower(metric) like '%resolutions%'
or lower(metric) like '%invoiced%'
or lower(metric) like '%retained%'
or lower(metric) like '%expected mrr%'
or lower(metric) like '%mrr%'
or lower(metric) like '%damage host%'
)
)