Merged PR 2624: Configuring by number of listings to prod
# Description Sets the parameter to display the KPIs by number of listings to prod. I will move forward without the review as we need a simultaneous deployment. The combination of changes were reviewed yesterday in local. # 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. Related work items: #19325
This commit is contained in:
parent
78838cb7f6
commit
3d09d04068
1 changed files with 2 additions and 2 deletions
|
|
@ -23,12 +23,12 @@ This configuration ensures that working with new dimensions won't affect the dis
|
||||||
until all development work has been done.
|
until all development work has been done.
|
||||||
Additionally, it provides a proper display name for reporting purposes.
|
Additionally, it provides a proper display name for reporting purposes.
|
||||||
|
|
||||||
{"dimension": "'by_number_of_listings'", "dimension_display": "'By # of Listings Booked in 12 Months'"}
|
|
||||||
*/
|
*/
|
||||||
|
|
||||||
{% 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'"}
|
||||||
] %}
|
] %}
|
||||||
{{ return(dimensions) }}
|
{{ return(dimensions) }}
|
||||||
{% endmacro %}
|
{% endmacro %}
|
||||||
Loading…
Add table
Add a link
Reference in a new issue