Merged PR 4596: Propagates Client Type to KPIs by Deal and AM models
# Description Propagates Client Type to KPIs by Deal and AM models. This is just a distinction between Platform and APIs. # 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: #28181
This commit is contained in:
parent
1a4a9fac37
commit
5d6e860e2b
8 changed files with 80 additions and 0 deletions
|
|
@ -354,6 +354,17 @@ models:
|
|||
data_tests:
|
||||
- not_null
|
||||
|
||||
- name: client_type
|
||||
data_type: string
|
||||
description: |
|
||||
Type of client. It can be either PLATFORM or API.
|
||||
data_tests:
|
||||
- not_null
|
||||
- accepted_values:
|
||||
values:
|
||||
- PLATFORM
|
||||
- API
|
||||
|
||||
- name: main_deal_name
|
||||
data_type: string
|
||||
description: |
|
||||
|
|
@ -441,6 +452,17 @@ models:
|
|||
data_tests:
|
||||
- not_null
|
||||
|
||||
- name: client_type
|
||||
data_type: string
|
||||
description: |
|
||||
Type of client. It can be either PLATFORM or API.
|
||||
data_tests:
|
||||
- not_null
|
||||
- accepted_values:
|
||||
values:
|
||||
- PLATFORM
|
||||
- API
|
||||
|
||||
- name: main_deal_name
|
||||
data_type: string
|
||||
description: |
|
||||
|
|
@ -1499,6 +1521,17 @@ models:
|
|||
- Previous 3 months
|
||||
- Previous month
|
||||
|
||||
- name: client_type
|
||||
data_type: string
|
||||
description: |
|
||||
Type of client. It can be either PLATFORM or API.
|
||||
data_tests:
|
||||
- not_null
|
||||
- accepted_values:
|
||||
values:
|
||||
- PLATFORM
|
||||
- API
|
||||
|
||||
- name: metric_from_date
|
||||
data_type: date
|
||||
description: |
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue