Fivetran's ad reporting dbt package. Combine your Facebook, Google, Pinterest, LinkedIn, Twitter, Snapchat, Microsoft, TikTok, Reddit, Amazon, and Apple Search advertising metrics using this package.
This PR will result in the following new package version:
v1.7.1
Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:
Bug Fixes
Adjust the severity of the account_id test in ad_reporting__account_report to warn. This is required since Snapchat can hard-delete records from the history tables, but not from the reporting tables. This ensures that accurate statistics are being reported and production pipelines aren't failing. (PR #20)
Please acknowledge that you have successfully performed the following commands locally:
[x] dbt run –full-refresh && dbt test
[ ] dbt run (if incremental models are present) && dbt test
Before marking this PR as "ready for review" the following have been applied:
[x] The appropriate issue has been linked, tagged, and properly assigned
[x] All necessary documentation and version upgrades have been applied
[x] docs were regenerated (unless this PR does not include any code or yml updates)
[x] BuildKite integration tests are passing
[x] Detailed validation steps have been provided below
❗ Special Updates for Ad Reporting ❗
To reduce integration testing time, not all models should be enabled in the run_models.sh vars. Update the variables after dbt run and dbt test to set:
[x] this PR's package to true
[x] Google Ads and Facebook Ads to true (if not already)
[x] All other packages to false
Detailed Validation
Please share any and all of your validation steps:
PR Overview
This PR will address the following Issue/Feature: https://github.com/fivetran/dbt_snapchat_ads/issues/16 (also contains merged PR https://github.com/fivetran/dbt_snapchat_ads/pull/20)
This PR will result in the following new package version:
v1.7.1
Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:
Bug Fixes
account_id
test inad_reporting__account_report
towarn
. This is required since Snapchat can hard-delete records from the history tables, but not from the reporting tables. This ensures that accurate statistics are being reported and production pipelines aren't failing. (PR #20)Under the Hood
Contributors
PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:
Before marking this PR as "ready for review" the following have been applied:
❗ Special Updates for Ad Reporting ❗
To reduce integration testing time, not all models should be enabled in the
run_models.sh
vars. Update the variables afterdbt run
anddbt test
to set:true
true
(if not already)false
Detailed Validation
Please share any and all of your validation steps:
See validation notes in https://github.com/fivetran/dbt_snapchat_ads/pull/24
If you had to summarize this PR in an emoji, which would it be?
🧪