Commit graph

1578 commits

Author SHA1 Message Date
Joaquin Ossa
2f484054df commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
3f224d8fed commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
9fa62e0bf2 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
29755961e6 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
47d9b6f292 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
7551f6e515 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
c89dd02799 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
271f017cf5 Merged PR 4482: Staging model of resolutions incidents
# Description

New staging model for Resolutions Incidents
It extracts all of the data from the incidents after the ingestion from CosmosDB.
This includes data on the host, the guest, the booking, the listing and all the resolution process until completed if completed.
Though it currently has a lot of problems like missing values, ids not existing in the back end or payments without currencies. Everything is more clarified here https://www.notion.so/truvi/Data-Quality-Report-Resolutions-Incidents-data-1a40446ff9c9807180eaccfeebb53045?pvs=4.

# 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: #27811
2025-02-25 14:15:27 +00:00
Joaquin Ossa
cace0f908d Merged PR 4507: Added timestamp to a/b test model
# Description

Added timestamp to a/b test model to start the measures from the adequate date and time

# 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.

Added timestamp to a/b test model

Related work items: #27721
2025-02-25 14:15:19 +00:00
Joaquin Ossa
1697bef090 timestamps 2025-02-25 15:14:54 +01:00
Oriol Roqué Paniagua
3c99c4f8bc Merged PR 4504: Onboarding MRR + Visualisation changes
# Description

Changes:
* Adds Onboarding MRR
* Refactors exclusion code for ongoing month / invoicing cycle
* Adds sign format on relative differences

# 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: #27609, #27805
2025-02-25 14:05:54 +00:00
Joaquin Ossa
03955aadd5 Finished schema 2025-02-25 14:54:59 +01:00
Joaquin Ossa
4a6158df28 Added timestamp to a/b test model 2025-02-25 14:48:37 +01:00
Oriol Roqué Paniagua
995027205c Merged PR 4502: Revert "Adds Onboarding MRR"
# Description

_Describe your motivation and changes here._

# 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.

Revert "Adds Onboarding MRR"

Reverted commit `dab210c6`.
2025-02-25 11:44:24 +00:00
Oriol Roqué Paniagua
7c4b0e752b Merged PR 4501: Revert "Refactor display exclusion for clarity purposes"
# Description

_Describe your motivation and changes here._

# 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.

Revert "Refactor display exclusion for clarity purposes"

Reverted commit `fdec616f`.
2025-02-25 11:43:46 +00:00
Oriol Roqué Paniagua
5eebb4f560 Merged PR 4500: Revert "Relative increments with sign"
# Description

_Describe your motivation and changes here._

# 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.

Revert "Relative increments with sign"

Reverted commit `f5718ded`.
2025-02-25 11:43:05 +00:00
uri
f5718ded4d Relative increments with sign 2025-02-25 12:30:23 +01:00
Joaquin Ossa
f036e626d7 Commit removed unnecessary fields 2025-02-25 12:19:45 +01:00
uri
fdec616fe6 Refactor display exclusion for clarity purposes 2025-02-25 11:45:14 +01:00
uri
dab210c607 Adds Onboarding MRR 2025-02-25 11:33:37 +01:00
Oriol Roqué Paniagua
a6671ee4d0 Merged PR 4493: Adds Revenue Churn Rate in YTD/MTD Overview
# Description

Changes:
* Adds Revenue Churn Rate in YTD/MTD Overview. This has several implications, I finally understood how to properly compute a YTD.

The problem is that Revenue Churn Rate is a % of the Total "Churned" Revenue in a 12 m period vs. the Total Revenue in the same 12 m period. This is a bit tricky because it's not really additive, because of the Churn definition. Total Churned Revenue is the Revenue that the churned deals in a month generated on that past 12 months prior to churning.

So - in order to aggregate it properly, we need to do the sum of the Total Churned Revenue and retrieve the Total Revenue on these 12 months, and THEN compute the Churn rate.

This PR mainly retrieves the necessary input from the Churn models and then follows a similar computation as for the rest of YTD/MTD converted metrics.

I'll handle Onboarding MRR in a separated PR as this one is quite dense already.

# 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: #27609, #27805
2025-02-25 09:41:28 +00:00
Joaquin Ossa
78d42462a8 Merged PR 4489: Update athena models to remove id_accommodation
# Description

Removed `id_accommodation` from athena models
I remembered why I had add it before, it was to unite it with the edeposit model with the same number of fields but I fixed it already

# 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.

Update athena models to remove id_accommodation

Related work items: #27938
2025-02-25 08:32:45 +00:00
Oriol Roqué Paniagua
fdc8e2ae24 Merged PR 4491: Reduce metric name length to fit in PBI
# Description

Reduce metric name length to fit in PBI

# 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.

Reduce metric name length to fit in PBI

Related work items: #27609
2025-02-24 16:40:01 +00:00
Joaquin Ossa
62316778fb updated schema 2025-02-24 16:35:10 +01:00
Joaquin Ossa
1d514cec3a Addressed comments 2025-02-24 16:27:16 +01:00
Joaquin Ossa
d7d3422eb9 Update athena models to remove id_accommodation 2025-02-24 15:21:44 +01:00
Joaquin Ossa
34f8fbe499 commit wip 2025-02-24 15:06:55 +01:00
Oriol Roqué Paniagua
85c2c73da7 Merged PR 4483: Adding targets (to be revised). Allows multi-year last-date computation
# Description

Changes:
* Adds template for targets -> These need to be revised
* Adds comparison of values vs. targets
* Allows for multi-year last date computation (allowing multiple years to be selected, not just the last one)

# 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: #27609, #27805
2025-02-24 10:39:45 +00:00
Joaquin Ossa
cbed87404e Merged PR 4481: Updated tests on S&P fees
# Description

Silly me added some `tests` instead of `data_tests`

# 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.

Updated tests on S&P fees
2025-02-24 10:38:03 +00:00
Joaquin Ossa
62c2e44b86 Deduplicated records 2025-02-24 10:57:46 +01:00
Joaquin Ossa
2e19678182 Final tests update 2025-02-23 16:39:43 +01:00
Joaquin Ossa
f29b67addd Updated tests on S&P fees 2025-02-23 16:14:42 +01:00
Joaquin Ossa
fb4775bd07 Updated tests 2025-02-23 16:11:37 +01:00
Joaquin Ossa
fc85dd9e70 Fixed data_types and added tests 2025-02-23 16:05:11 +01:00
Joaquin Ossa
d6dcbcc67e commit wip 2025-02-23 15:04:59 +01:00
Joaquin Ossa
f3842c950b commit wip 2025-02-22 19:55:30 +01:00
Joaquin Ossa
9704426dd6 commit wip 2025-02-22 14:16:41 +01:00
Joaquin Ossa
05adc07674 commit wip 2025-02-21 17:44:00 +01:00
Joaquin Ossa
1b853a9503 commit wip (id_incident duplicates have to check) 2025-02-21 16:40:53 +01:00
Joaquin Ossa
725f2a54c3 Staging model for resolution incidents 2025-02-21 15:25:24 +01:00
Oriol Roqué Paniagua
8cf7393173 Merged PR 4471: Visualisation changes
# Description

Adapts YTD/MTD for visualisation purposes

# 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.

Visualisation changes

Related work items: #27609
2025-02-20 18:43:26 +00:00
Joaquin Ossa
5942e4b81c Merged PR 4463: Update exposures
# Description

Update exposures of new and updated reports

# Checklist

- [x] The edited models and dependants run properly with production data.**dbt compile completed**
- [ ] 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.

Update exposures
2025-02-20 13:43:21 +00:00
Joaquin Ossa
aed9ce5575 Update exposures 2025-02-20 14:33:58 +01:00
Joaquin Ossa
f28db85968 Merged PR 4455: Added has_verification_request
# Description

Added has_verification_request for New Dash report

# 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.

Added has_verification_request

Related work items: #27787
2025-02-19 15:39:04 +00:00
Joaquin Ossa
b35d9b0a00 Addressed comments 2025-02-19 16:37:40 +01:00
Joaquin Ossa
c710de7117 Added has_verification_request 2025-02-19 16:00:33 +01:00
Joaquin Ossa
361342af98 Merged PR 4454: Added listings data
# Description

I had to include new fields on the model since Power BI couldn't handle 1 join, apparently it was too much data for it

# 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.

Added listings data

Related work items: #27771
2025-02-19 14:32:18 +00:00
Joaquin Ossa
a94fb760e5 new booking fields 2025-02-19 15:29:05 +01:00
Joaquin Ossa
6fce8c0ec6 Added listings data 2025-02-19 14:57:44 +01:00
Joaquin Ossa
5585880b63 Merged PR 4448: Added booking id to payments model
# Description

Added booking id to payments model to upgrade payments report

# 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.

Added booking id to payments model

Related work items: #27771
2025-02-19 10:59:13 +00:00