After discussion with Pablo on the fact that Deposits are only with status "Paid" for a given time before they get Cancelled or Refunded, we just believe it's best to remove the Deposits amount from the Guest Payments metric. In any case, this does not represent a Revenue source... This was discovered while doing the data quality assessment for revenue figures ([here](https://www.notion.so/knowyourguest-superhog/Data-quality-assessment-DWH-vs-Finance-revenue-figures-6e3d6b75cdd4463687de899da8aab6fb)) Before, `total_guest_payments_in_gbp` was a standalone metric that computed any payment by the guest with status paid. We were computing revenue based on the `total_guest_income_in_gbp`, which mainly was the sum of waiver payments, deposit fees (not deposit itself!) and check in hero fees. Mainly what I did is: - remove the existing `total_guest_payments_in_gbp` in the source models (int_core__xxx_guest_payments_xxx) - rename the already existing `total_guest_income_in_gbp` to `total_guest_payments_in_gbp` Related work items: #18787, #18914 |
||
|---|---|---|
| .. | ||
| int_daily_currency_exchange_rates.sql | ||
| int_dates.sql | ||
| int_dates_by_deal.sql | ||
| int_dates_mtd.sql | ||
| int_monthly_aggregated_metrics_history_by_deal.sql | ||
| int_mtd_aggregated_metrics.sql | ||
| int_mtd_vs_previous_year_metrics.sql | ||
| int_simple_exchange_rates.sql | ||
| schema.yml | ||