Merged PR 4018: PMS data to reporting

# Description

PMS data to reporting

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

PMS data to reporting

Related work items: #26082
This commit is contained in:
Joaquin Ossa 2025-01-10 08:57:07 +00:00
commit 1217c92337
2 changed files with 15 additions and 0 deletions

View file

@ -7,6 +7,8 @@ select
id_deal as id_deal, id_deal as id_deal,
main_deal_name as main_deal_name, main_deal_name as main_deal_name,
has_active_pms as has_active_pms,
active_pms_list as active_pms_list,
main_billing_country_iso_3_per_deal as main_billing_country_iso_3_per_deal, main_billing_country_iso_3_per_deal as main_billing_country_iso_3_per_deal,
deal_lifecycle_state as deal_lifecycle_state, deal_lifecycle_state as deal_lifecycle_state,
deal_hubspot_stage as deal_hubspot_stage, deal_hubspot_stage as deal_hubspot_stage,

View file

@ -593,6 +593,19 @@ models:
tests: tests:
- not_null - not_null
- name: has_active_pms
data_type: boolean
description: |
Does the deal have an active associated PMS.
tests:
- not_null
- name: active_pms_list
data_type: string
description: |
Name of the active PMS associated with the deal. It can have more than
one PMS associated with it. It can be null if it doesn't have any PMS associated.
- name: main_billing_country_iso_3_per_deal - name: main_billing_country_iso_3_per_deal
data_type: string data_type: string
description: | description: |