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

Merged PR 2755: Exposes By Billing Country dimension in KPIs to production

Browse source 
# Description

Parameter configuration to show the new dimension now that tests are implemented and look ok.

# 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.
- [X] I have checked for DRY opportunities with other models and docs.
- [X] 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: #19082, #20827
This commit is contained in:
Oriol Roqué Paniagua 2024-09-05 14:55:30 +00:00 • committed by Pablo Martín
commit 00a97a3aff
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
macros/business_kpis_configuration.sql
View file

@ -29,7 +29,8 @@ Additionally, it provides a proper display name for reporting purposes.
{% macro get_kpi_dimensions_for_production() %} {% macro get_kpi_dimensions_for_production() %}
{% set dimensions = [ {% set dimensions = [
{"dimension": "'global'", "dimension_display": "'Global'"}, {"dimension": "'global'", "dimension_display": "'Global'"},
{"dimension": "'by_number_of_listings'", "dimension_display": "'By # of Listings Booked in 12 Months'"} {"dimension": "'by_number_of_listings'", "dimension_display": "'By # of Listings Booked in 12 Months'"},
{"dimension": "'by_billing_country'", "dimension_display": "'By Billing Country'"}
] %} ] %}
{{ return(dimensions) }} {{ return(dimensions) }}
{% endmacro %} {% endmacro %}