Merged PR 4130: Propagate aggregations to xero sales reporting

# Description

Propagates the xero aggregations to the reporting model of xero__sales_denom_mart

# 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: #26618
This commit is contained in:
Oriol Roqué Paniagua 2025-01-21 13:22:36 +00:00
parent a99b7f29a7
commit ddc806583c
2 changed files with 20 additions and 2 deletions

View file

@ -1364,3 +1364,19 @@ models:
- name: contact_name
data_type: character varying
description: ""
- name: accounting_root_aggregation
data_type: character varying
description: |
The root aggregation per account code. This is the main
aggregation that is used to retrieve low-level data.
- name: accounting_kpis_aggregation
data_type: character varying
description: |
The default macro-aggregation for Invoiced KPIs.
- name: accounting_financial_aggregation
data_type: character varying
description: |
The default macro-aggregation for Financial reporting.

View file

@ -39,6 +39,8 @@ select
sdm.header_total_due_in_gbp as header_total_due_in_gbp,
sdm.id_contact as id_contact,
sdm.id_deal as id_deal,
sdm.contact_name as contact_name
sdm.contact_name as contact_name,
sdm.accounting_root_aggregation as accounting_root_aggregation,
sdm.accounting_kpis_aggregation as accounting_kpis_aggregation,
sdm.accounting_financial_aggregation as accounting_financial_aggregation
from int_xero__sales_denom_mart sdm