Releases/v0.6.latest

[fivetran-jamie]

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:

v0.6.2

Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:

Bug Fixes

• Adjust the severity of the ad_account_id test in snapchat_ads__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)
  • Documents the above decision in the DECISIONLOG.md.

Under the Hood

• Updated the repo-maintainer PR template to our most up-to-date format.

Contributors

• @bthomson22 (PR #20)

PR Checklist

Basic Validation

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

Detailed Validation

Please share any and all of your validation steps:

Confirmed working by @bthomson22, but i also recreated the issue in a somewhat hacky way by

1. running the package as normal

2. updating the stg_snapchat_ads__ad_account_history to have null account IDs

3. running dbt run -m snapchat_ads__account_report without the code update

4. dbt test -m snapchat_ads__account_report -- produces failure without adjusting severity

5. dbt test again after updating the not_null test to be a warning (did so in Ad Reporting as well)

If you had to summarize this PR in an emoji, which would it be?

⚠️