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

Merged PR 3855: fixed number of nights in edeposit

Browse source 
# Description

There was an error in the number of nights for e-deposit verifications because the order of the columns when using union all of both athena and e-deposit tables that `number_of_night`s was switched with `total_fee_local` for e-deposit, this was generating weird values that was catched on by Ana.

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

fixed number of nights in edeposit

Related work items: #25546
This commit is contained in:
Joaquin Ossa 2024-12-16 13:58:49 +00:00
commit defa36de8a
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
models/intermediate/cross/int_edeposit_and_athena_verifications.sql
View file

@ -12,12 +12,12 @@ select
av.verification_source,
av.verification_status,
av.nightly_fee_local,
av.number_nights,
case
when av.number_nights = 0
then av.nightly_fee_local
else av.nightly_fee_local * av.number_nights
end as total_fee_local,
av.number_nights,
av.email_flag,
av.phone_flag,
av.watch_list,