Commit graph

17 commits

Author SHA1 Message Date
Pablo Martín
a24c2efb5a Merged PR 2631: Accounting Report Exposures
# Description

Creates exposure entries for the accounting reports that are already in production (Invoicing and Crediting and Resolutions Host Payments).

# Checklist

- __NA__ ~~[ ] The edited models and dependants run properly with production data.~~
- [X] The edited models are sufficiently documented.
- __NA__ ~~[ ] The edited models contain PK tests, and I've ran and passed them.~~
- __NA__ ~~[ ] I have checked for DRY opportunities with other models and docs.~~
- __NA__ ~~[ ] 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: #17550
2024-08-22 12:23:14 +00:00
Oriol Roqué Paniagua
0c1535b150 Merged PR 2267: Business KPIs refactor - cleaning
This PR cleans the unused models in reporting that keep the "core" wrong convention. Also drops schema documentation for these.
Lastly, exposures have been updated to point towards the new production models.

- This is the last PR for the refactor! ![dbt-dag (2).png](https://guardhog.visualstudio.com/4148d95f-4b6d-4205-bcff-e9c8e0d2ca65/_apis/git/repositories/54ac356f-aad7-46d2-b62c-e8c5b3bb8ebf/pullRequests/2267/attachments/dbt-dag%20%282%29.png)

Refactor documentation: https://www.notion.so/knowyourguest-superhog/Refactoring-Business-KPIs-5deb6aadddb34884ae90339402ac16e3

Related work items: #18202
2024-07-11 09:17:44 +00:00
Joaquin Ossa
9a74e21783 Fixed description 2024-07-11 09:53:01 +02:00
Joaquin Ossa
f96587dc33 Added report to exposure.yaml 2024-07-11 09:07:22 +02:00
Joaquin Ossa
2efc1d8b65 Merged PR 2178: New model for guests satisfaction report
New model for guests satisfaction report, I included columns to check what is the guest paying for that might be helpful for analysis as well

Related work items: #16947
2024-07-04 10:13:11 +00:00
Joaquin Ossa
5867b53e55 Fixed indentation 2024-07-03 15:01:07 +02:00
Joaquin Ossa
af4ab70b96 Addresed Uri's comments and also included new reports in exposures.ymal 2024-07-03 12:29:01 +02:00
Oriol Roqué Paniagua
ed5d7828a7 Merged PR 2179: Computes aggregated metrics by deal id and exposes it to reporting
This PR creates 2 new models:
- `int_core__monthly_aggregated_metrics_history_by_deal`, which just gathers the information of the previously created models that compute the kpis by deal id.
- `core__monthly_aggregated_metrics_history_by_deal`, effectively a copy from intermediate to reporting

It also includes documentation of these 2 models, differences between these and the `mtd_aggregated_metrics` equivalents and references it to exposures. I took the opportunity to update the documentation of the `core__mtd_aggregated_metrics` now that it's a bit more mature.

This should be the last PR for the first draft of 'by deal' metrics.

Related work items: #17689
2024-07-03 07:06:34 +00:00
uri
c8807ddf60 Adding exposure of main kpis dashboard 2024-06-17 14:04:51 +02:00
Oriol Roqué Paniagua
3610c447f5 Merged PR 1949: Fix documentation in DWH and include exposure
Related work items: #16857
2024-05-30 10:37:41 +00:00
Pablo Martin
e2645814e4 update exposures 2024-05-23 16:15:43 +02:00
Pablo Martin
751219a768 add dependency on exposure 2024-05-15 19:11:28 +02:00
Pablo Martin
9b9e5c32b5 missing exposure 2024-05-07 17:55:03 +02:00
Pablo Martin
18b319de00 add exposure 2024-05-06 17:52:42 +02:00
Pablo Martin
d7b810df73 Add host booking fees exposure. 2024-04-17 17:15:03 +02:00
Pablo Martin
68ddbf23c6 add link to pbi exposure 2024-03-15 11:58:02 +01:00
Pablo Martin
689d5eff16 docs 2024-03-13 15:47:58 +01:00