Merged PR 3490: Removal of other_sharing_platform_url

# Description

Removes other_sharing_platform_url field that will disappear from the backend in the release going out next week.

Apparently there's a dependency with a PBI report that uses core__unified_user table, so I just nullified the reporting field. The report is called users_dashboard, which AFAIK is not published anywhere. I wonder if this is one of the it-should-not-exist report. Waiting for tomorrow confirmation with Pablo.

Important note: Since this removes fields from staging, which is incremental, we need to trigger a full-refresh for this table. Otherwise execution will fail.

# Checklist

- [X] The edited models and dependants run properly with production data.
- [NA] The edited models are sufficiently documented.
- [X] 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

- [X] Check if a full-refresh is required after this PR is merged. **Yes, please refer to my previous comment.**

Related work items: #23974
This commit is contained in:
Oriol Roqué Paniagua 2024-11-13 09:23:42 +00:00
parent c21040fa30
commit cce5d2b9d6
3 changed files with 2 additions and 5 deletions

View file

@ -22,8 +22,6 @@ with
{{ adapter.quote("NumberOfProperties") }} as number_of_properties,
{{ adapter.quote("SuperhogVerifiedSetId") }} id_superhog_verified_set,
{{ adapter.quote("PlatformCommsRecipient") }} as platform_comms_recipient,
{{ adapter.quote("OtherSharingPlatformUrl") }}
as other_sharing_platform_url,
{{ adapter.quote("UserVerificationStatusId") }}
as id_user_verification_status,
{{ adapter.quote("_airbyte_extracted_at") }} as dwh_loaded_at_date