Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Research ability to add Passthrough Columns #8

Closed
kristin-bagnall opened this issue Jan 6, 2021 · 5 comments
Closed

Research ability to add Passthrough Columns #8

kristin-bagnall opened this issue Jan 6, 2021 · 5 comments
Assignees

Comments

@kristin-bagnall
Copy link
Contributor

The pin_promotion_report table includes additional metrics (other than spend, impressions, and clicks) that will be helpful to include in the pinterest_ads__ad_adapter model.

Determine whether or not it is feasible to allow users to add in additional metrics that will then be aggregated in the final model.

I've started testing out this functionality in the passthrough_variable branch.

@kristin-bagnall
Copy link
Contributor Author

Before we do this, one thing we'll need to do some discovery on is:

  • What are some example set of columns customers have in the pin_promotion_report table
  • How will this impact the ad_reporting roll up package that is set to release shortly

@arniwesth
Copy link

Thanks for looking into this possibility. In my particular case, I need to extract checkout conversions from pin_promotion_report.
These are the relevant fields:

total_checkout_desktop_action_to_desktop_conversion
total_checkout_desktop_action_to_mobile_conversion
total_checkout_desktop_action_to_mobile_conversion
total_checkout_mobile_action_to_desktop_conversion
total_checkout_mobile_action_to_mobile_conversion
total_checkout_tablet_action_to_desktop_conversion
total_checkout_tablet_action_to_mobile_conversion
total_checkout_tablet_action_to_tablet_conversion

@kristin-bagnall
Copy link
Contributor Author

Hey @arniwesth - we're evaluating whether to add this to our next sprint. I'm curious, is this still a priority for you, or have you found a workaround?

@arniwesth
Copy link

Hey @fivetran-kristin - I found a workaround where we can still utilize the Fivetran dbt package and add custom metrics as we need. So not a high priority from our perspective.

@fivetran-joemarkiewicz
Copy link
Contributor

Closing this Issue as the previously mentioned PRs have been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants