Merged PR 2933: Financial rate metrics
# Description Changed the financial rate metrics to have `number_fomat = "converted_metric_currency_gbp"` # 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. - [ ] 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. Financial rate metrics Related work items: #21564
This commit is contained in:
commit
e401efdd5d
4 changed files with 25 additions and 16 deletions
|
|
@ -239,7 +239,7 @@
|
|||
"value": "total_revenue_per_created_booking",
|
||||
"previous_year_value": "previous_year_total_revenue_per_created_booking",
|
||||
"relative_increment": "relative_increment_total_revenue_per_created_booking",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -248,7 +248,7 @@
|
|||
"value": "total_revenue_per_created_guest_journey",
|
||||
"previous_year_value": "previous_year_total_revenue_per_created_guest_journey",
|
||||
"relative_increment": "relative_increment_total_revenue_per_created_guest_journey",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -257,7 +257,7 @@
|
|||
"value": "total_revenue_per_deals_booked_in_month",
|
||||
"previous_year_value": "previous_year_total_revenue_per_deals_booked_in_month",
|
||||
"relative_increment": "relative_increment_total_revenue_per_deals_booked_in_month",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -266,7 +266,7 @@
|
|||
"value": "total_revenue_per_listings_booked_in_month",
|
||||
"previous_year_value": "previous_year_total_revenue_per_listings_booked_in_month",
|
||||
"relative_increment": "relative_increment_total_revenue_per_listings_booked_in_month",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -347,7 +347,7 @@
|
|||
"value": "guest_revenue_per_completed_guest_journey",
|
||||
"previous_year_value": "previous_year_guest_revenue_per_completed_guest_journey",
|
||||
"relative_increment": "relative_increment_guest_revenue_per_completed_guest_journey",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -356,7 +356,7 @@
|
|||
"value": "guest_revenue_per_paid_guest_journey",
|
||||
"previous_year_value": "previous_year_guest_revenue_per_paid_guest_journey",
|
||||
"relative_increment": "relative_increment_guest_revenue_per_paid_guest_journey",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -428,7 +428,7 @@
|
|||
"value": "guest_payments_per_completed_guest_journey",
|
||||
"previous_year_value": "previous_year_guest_payments_per_completed_guest_journey",
|
||||
"relative_increment": "relative_increment_guest_payments_per_completed_guest_journey",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
{
|
||||
|
|
@ -437,7 +437,7 @@
|
|||
"value": "guest_payments_per_paid_guest_journey",
|
||||
"previous_year_value": "previous_year_guest_payments_per_paid_guest_journey",
|
||||
"relative_increment": "relative_increment_guest_payments_per_paid_guest_journey",
|
||||
"number_format": "currency_gbp",
|
||||
"number_format": "converted_metric_currency_gbp",
|
||||
"increment_sign_format": "positive",
|
||||
},
|
||||
] %}
|
||||
|
|
|
|||
|
|
@ -421,7 +421,13 @@ models:
|
|||
description: allows for grouping and formatting for displaying purposes.
|
||||
tests:
|
||||
- accepted_values:
|
||||
values: ["integer", "percentage", "currency_gbp"]
|
||||
values:
|
||||
[
|
||||
"integer",
|
||||
"percentage",
|
||||
"currency_gbp",
|
||||
"converted_metric_currency_gbp",
|
||||
]
|
||||
|
||||
- name: value
|
||||
data_type: numeric
|
||||
|
|
|
|||
|
|
@ -3,7 +3,7 @@
|
|||
with
|
||||
dimensions as (
|
||||
{% for dimension in production_dimensions %}
|
||||
select
|
||||
select
|
||||
{{ dimension.dimension }} as dimension,
|
||||
{{ dimension.dimension_display }} as dimension_display
|
||||
{% if not loop.last %}
|
||||
|
|
@ -12,15 +12,12 @@ with
|
|||
{% endfor %}
|
||||
),
|
||||
int_mtd_aggregated_metrics as (
|
||||
select
|
||||
m.*,
|
||||
d.dimension_display
|
||||
select m.*, d.dimension_display
|
||||
from {{ ref("int_mtd_aggregated_metrics") }} m
|
||||
-- The following clause limits the display execution
|
||||
-- to only include those dimensions configured to
|
||||
-- appear for production purposes
|
||||
inner join dimensions d
|
||||
on m.dimension = d.dimension
|
||||
inner join dimensions d on m.dimension = d.dimension
|
||||
)
|
||||
|
||||
select
|
||||
|
|
|
|||
|
|
@ -405,7 +405,13 @@ models:
|
|||
description: Allows for grouping and formatting for displaying purposes.
|
||||
tests:
|
||||
- accepted_values:
|
||||
values: ["integer", "percentage", "currency_gbp"]
|
||||
values:
|
||||
[
|
||||
"integer",
|
||||
"percentage",
|
||||
"currency_gbp",
|
||||
"converted_metric_currency_gbp",
|
||||
]
|
||||
|
||||
- name: value
|
||||
data_type: numeric
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue