counterweight/data-dwh-dbt-project
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merged PR 4020: Expose new KPIs

Browse source 
# Description

Exposes 2 New KPIs on Revenue Retained. Work was already done, I just removed the temporary filter and adapted the name of the metric.

# 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.

Expose new KPIs

Related work items: #25804
This commit is contained in:
Oriol Roqué Paniagua 2025-01-10 10:07:00 +00:00
parent 1217c92337
commit 6ace9230e9
2 changed files with 1 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
models/intermediate/cross/int_mtd_aggregated_metrics.sql
View file

@ -271,7 +271,7 @@
}, },
{ {
"order_by": 202, "order_by": 202,
"metric": "Revenue Retained Post Resolutions", "metric": "Revenue Retained Post-Resolutions",
"value": "revenue_retained_post_resolutions_in_gbp", "value": "revenue_retained_post_resolutions_in_gbp",
"previous_year_value": "previous_year_revenue_retained_post_resolutions_in_gbp", "previous_year_value": "previous_year_revenue_retained_post_resolutions_in_gbp",
"relative_increment": "relative_increment_revenue_retained_post_resolutions_in_gbp", "relative_increment": "relative_increment_revenue_retained_post_resolutions_in_gbp",

3
models/reporting/general/mtd_aggregated_metrics.sql
View file

@ -84,6 +84,3 @@ where
) )
-- If metric is Churn Rate, do not show month in progress -- If metric is Churn Rate, do not show month in progress
and not (lower(metric) like '%churn rate%' and is_current_month = true) and not (lower(metric) like '%churn rate%' and is_current_month = true)
-- Temporary exclude Income Retained metrics until we have a proper naming
-- convention in place
and not (lower(metric)) like '%revenue retained%'