As noted by @FlorianASchroeder , PR #250 allowed the union of intraday data only for single-property configurations.
This update removes that limitation by leveraging the fact that both daily and intraday event tables start with 'events'. This means we can use a wildcard operation to query `events*to pull both daily and intraday records. Duplicate records will be deduped as part of thebase_ga4__events` qualify statement.
As a result, we can remove the intraday source from src_ga4.yml
Checklist
[x] I have verified that these changes work locally
[na] I have updated the README.md (if applicable)
[na] I have added tests & descriptions to my models (and macros if applicable)
[x] I have run dbt test and python -m pytest . to validate existing tests
Description & motivation
Resolves #254
As noted by @FlorianASchroeder , PR #250 allowed the union of intraday data only for single-property configurations.
This update removes that limitation by leveraging the fact that both daily and intraday event tables start with 'events'. This means we can use a wildcard operation to query `events*
to pull both daily and intraday records. Duplicate records will be deduped as part of thebase_ga4__events` qualify statement.As a result, we can remove the intraday source from
src_ga4.ymlChecklist
dbt testandpython -m pytest .to validate existing tests