counterweight/data-dwh-dbt-project
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merged PR 3597: Adds new API roles as hosts

Browse source 
# Description

This fixes the data alert raised in today execution. We have the first user with the role `ScreenAndProtectApi`.

Changes:
* Adds `CancellationApi`, `CheckInHeroApi` and `ScreenAndProtectApi` as accepted roles for `int_core__user_host` computation
* Adds the new `ScreenAndProtectApi` role as accepted values in the `int_core__user_role` model

# 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: #24475
This commit is contained in:
Oriol Roqué Paniagua 2024-11-20 09:02:22 +00:00
parent 23cc5459e3
commit 23105d9f1b
2 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
models/intermediate/core/int_core__user_host.sql
View file

@ -1,4 +1,4 @@
{% set host_roles = "('EDEPOSIT', 'HOST', 'KNOWYOURGUEST', 'PLATFORM', 'PROPERTYVERIFICATIONMANAGER', 'SCREENINGAPI')" %} {% set host_roles = "('EDEPOSIT', 'HOST', 'KNOWYOURGUEST', 'PLATFORM', 'PROPERTYVERIFICATIONMANAGER', 'SCREENINGAPI', 'CANCELLATIONAPI', 'CHECKINHEROAPI','SCREENANDPROTECTAPI')" %}
{% set kyg_claim_types = "('KYGREGISTRATIONSIGNUPTYPE','KYGREGISTRATIONINTEGRATIONTYPENAME','KYGMVP', 'NEWDASHVERSION')" %} {% set kyg_claim_types = "('KYGREGISTRATIONSIGNUPTYPE','KYGREGISTRATIONINTEGRATIONTYPENAME','KYGMVP', 'NEWDASHVERSION')" %}
{{ config(materialized="table", unique_key="id_user_host") }} {{ config(materialized="table", unique_key="id_user_host") }}

6
models/intermediate/core/schema.yml
View file

@ -1692,6 +1692,7 @@ models:
- PropertyVerificationManager - PropertyVerificationManager
- CheckInHeroApi - CheckInHeroApi
- CancellationApi - CancellationApi
- ScreenAndProtectAPI
tests: tests:
- dbt_utils.unique_combination_of_columns: - dbt_utils.unique_combination_of_columns:
@ -1723,6 +1724,7 @@ models:
- PropertyVerificationManager - PropertyVerificationManager
- CheckInHeroApi - CheckInHeroApi
- CancellationApi - CancellationApi
- ScreenAndProtectApi
- name: int_core__user_host - name: int_core__user_host
description: | description: |
@ -1738,10 +1740,14 @@ models:
- KnowYourGuest - KnowYourGuest
- ScreeningAPI - ScreeningAPI
- PropertyVerificationManager - PropertyVerificationManager
- CancellationApi
- CheckInHeroApi
- ScreenAndProtectApi
Additionally, any user that has any of these claim types will be considered as a Host: Additionally, any user that has any of these claim types will be considered as a Host:
- KygRegistrationSignUpType - KygRegistrationSignUpType
- KygRegistrationIntegrationTypeName - KygRegistrationIntegrationTypeName
- KygMvp - KygMvp
- NewDashVersion
Lastly, in case a user satisfies multiple conditions, it will only appear once in this table. Lastly, in case a user satisfies multiple conditions, it will only appear once in this table.
columns: columns: