Merged PR 4352: Bulk update dash_source to business_scope
# Description Changes: * Switches dash_source to business_scope in all kpis models, schema and macro configuration. # 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: #27356
This commit is contained in:
parent
4f2a34427f
commit
78707ef649
23 changed files with 231 additions and 164 deletions
|
|
@ -56,8 +56,12 @@ Please note that strings should be encoded with " ' your_value_here ' ",
|
|||
{% macro dim_deal() %}
|
||||
{{ return({"dimension": "'by_deal'", "dimension_value": "id_deal"}) }}
|
||||
{% endmacro %}
|
||||
{% macro dim_dash() %}
|
||||
{{ return({"dimension": "'by_dash_source'", "dimension_value": "dash_source"}) }}
|
||||
{% macro dim_business_scope() %}
|
||||
{{
|
||||
return(
|
||||
{"dimension": "'by_business_scope'", "dimension_value": "business_scope"}
|
||||
)
|
||||
}}
|
||||
{% endmacro %}
|
||||
{% macro dim_has_payment() %}
|
||||
{{ return({"dimension": "'by_has_payment'", "dimension_value": "has_payment"}) }}
|
||||
|
|
@ -140,7 +144,7 @@ Provides a general assignement for the Dimensions available for each KPI
|
|||
"GUEST_PAYMENTS",
|
||||
"STARTED_GUEST_JOURNEYS",
|
||||
] %}
|
||||
{% set additional_dimensions = additional_dimensions + [dim_dash()] %}
|
||||
{% set additional_dimensions = additional_dimensions + [dim_business_scope()] %}
|
||||
{% endif %}
|
||||
|
||||
{% if entity_name == "CHECK_IN_ATTRIBUTED_GUEST_JOURNEYS" %}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue