Merged PR 5422: Change if block to avoid whitespace trimming
# Description Model `int_core__vr_check_in_cover` is producing invalid SQL because of a typo: a Jinja `if` block gets created with whitespace trimming, so the output gets concatenated to the previous code without whitespace and the resulting SQL is crap. This PR changes the if block so it doesn't trim whitespace, hence producing valid SQL and the indended query. # 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. - [ ] 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: #30165
This commit is contained in:
commit
58df9fbe87
1 changed files with 2 additions and 3 deletions
|
|
@ -225,13 +225,12 @@ with
|
||||||
|
|
||||||
select *
|
select *
|
||||||
from pre_feature_flag_verification_requests
|
from pre_feature_flag_verification_requests
|
||||||
|
{% if modules.datetime.datetime.now(
|
||||||
{%- if modules.datetime.datetime.now(
|
|
||||||
modules.pytz.timezone("UTC")
|
modules.pytz.timezone("UTC")
|
||||||
) > modules.datetime.datetime.strptime(
|
) > modules.datetime.datetime.strptime(
|
||||||
var("guest_products_feature_flag_activation_timestamp"),
|
var("guest_products_feature_flag_activation_timestamp"),
|
||||||
"'%Y-%m-%dT%H:%M:%S%z'",
|
"'%Y-%m-%dT%H:%M:%S%z'",
|
||||||
) -%}
|
) %}
|
||||||
union all
|
union all
|
||||||
select *
|
select *
|
||||||
from post_feature_flag_verification_requests
|
from post_feature_flag_verification_requests
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue