Feature/conversions

[fivetran-reneeli]

PR Overview

This PR will address the following Issue/Feature:
introduce conversion metrics

This PR will result in the following new package version:

Breaking change. Adding new fields

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

dbt_reddit_ads_source v0.3.0

PR #7 includes the following updates:

Feature: Conversion Metrics

• Introduces 4 new staging models to bring in conversion metrics across different grains:
  • stg_reddit_ads__account_conversions_report
  • stg_reddit_ads__ad_group_conversions_report
  • stg_reddit_ads__ad_conversions_report
  • stg_reddit_ads__campaign_conversions_report
• These models bring in conversions (click-through conversions) and view-through conversions, in addition to total items and total value, for each day and respective grain.
• We use the maximum attribution window when considering conversions and therefore retrieve conversions metrics from the click_through_conversion_attribution_window_month and view_through_conversion_attribution_window_month fields in the respective source tables. You may bring in additional windows and fields through the stg_<>_conversions_report models via the <>_conversions_passthrough_metrics variable. For information on how to configure these variables, refer to the README.

Under the hood

• Added the respective seed data for the new models in addition to updating relevant documentation.
• Added documentation explaining potential discrepancies across reporting grains.

PR Checklist

Basic Validation

Please acknowledge that you have successfully performed the following commands locally:

• dbt run –full-refresh && dbt test
• [na] dbt run (if incremental models are present) && dbt test

Before marking this PR as "ready for review" the following have been applied:

• [na] The appropriate issue has been linked, tagged, and properly assigned
• All necessary documentation and version upgrades have been applied
• docs were regenerated (unless this PR does not include any code or yml updates)
• BuildKite integration tests are passing
• Detailed validation steps have been provided below

Detailed Validation

Please share any and all of your validation steps:

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

💃

[fivetran-reneeli]

will close out issue #3 as well since this is addressed

[fivetran-joemarkiewicz]

@fivetran-jamie generally this PR looks good. There are a few changes I would like to be addressed before this is ready for approval. Let me know if you have any questions.

[fivetran-joemarkiewicz]

LGTM