[Bug] Add correct variable name (passthrough metric) in `ad_reporting__url_report`

[aleix-cd]

Is there an existing issue for this?

• I have searched the existing issues

Describe the issue

When developing #84, I forgot to change the variable name in ad_reporting__url_report.

We decided, together with Joe, that we would keep the same variable name for both Ad and URL reports, but I never changed the variable name in the URL report (see models/ad_reporting__url_report.sql):

{{ fivetran_utils.persist_pass_through_columns(pass_through_variable='ad_reporting__url_passthrough_metrics', transform = 'sum') }}

Instead, it should be:

{{ fivetran_utils.persist_pass_through_columns(pass_through_variable='ad_reporting__ad_passthrough_metrics', transform = 'sum') }}

Relevant error log or model output

No response

Expected behavior

I would expect, when using the ad_reporting__ad_passthrough_metrics variable, for the model ad_reporting__url_report to include any metric specified in that variable when compiled.

dbt Project configurations

google_ads__ad_stats_passthrough_metrics:
    - name: conversions
linkedin_ads__creative_passthrough_metrics:
    - name: external_website_conversions
      alias: conversions
microsoft_ads__ad_passthrough_metrics:
    - name: conversions
facebook_ads__basic_ad_passthrough_metrics:
    - name: conversions
ad_reporting__ad_passthrough_metrics:
    - name: conversions

Package versions

- package: fivetran/ad_reporting
  version: [">=1.4.0", "<1.5.0"]

What database are you using dbt with?

snowflake

dbt Version

v1.5.2

Additional Context

No response

Are you willing to open a PR to help address this issue?

• Yes.
• Yes, but I will need assistance and will schedule time during our office hours for guidance
• No.

[fivetran-joemarkiewicz]

This has since been resolved within the latest v.1.5.0 release of the package. As such, I will close this ticket out. Thanks @aleix-cd for catching this and opening your PR to address the issue!