[Feature] Update to use `fivetran_utils.extract_url_parameter` instead of `dbt_utils.get_url_parameter`
Opened this issue · 1 comments
Copied from fivetran/dbt_facebook_ads #37.
Is there an existing feature request for this?
- I have searched the existing issues
Describe the Feature
An issue with dbt_utils.get_url_parameter was discovered in fivetran/dbt_linkedin_source#55 for Databricks SQL. Macro fivetran_utils.extract_url_parameter was added to solve this issue for Databricks targets (see fivetran/dbt_fivetran_utils#130 more info).
For this package, references to dbt_utils.get_url_parameter need to be updated to fivetran_utils.extract_url_parameter.
The latest release of this package has been updated to use the custom google_ads_source.extract_url_parameter() macro. This addresses the immediate issue raised in this feature request. However, in the next breaking change of this package we will want to remove the custom macro and swap the package to leverage the fivetran_utils.extract_url_parameter() macro.
This was not implemented originally as it could possibly have caused breaking changes, we wanted to address the issue in a timely manner while ensuring we did not break any transformation pipelines.