[Bug] Small typo in boolean logic for config: enabled

[jtcohen6]

Is there an existing issue for this?

• I have searched the existing issues

Describe the issue

This results in "False and True", which is not a valid boolean value:

dbt_amazon_ads_source/models/src_amazon_ads.yml

Lines 167 to 168 in ba849f3

	config:
	enabled: "{{ var('ad_reporting__amazon_ads_enabled', true) }} and {{ var('amazon_ads__portfolio_history_enabled', true) }}"

This results in an admittedly gross validation error in dbt-core:

hologram.ValidationError: 'False and True' is not of type 'boolean'

Failed validating 'type' in schema['properties']['enabled']:
    {'default': True, 'type': 'boolean'}

On instance['enabled']:
    'False and True'

Change to:

 config: 
   enabled: "{{ var('ad_reporting__amazon_ads_enabled', true) and var('amazon_ads__portfolio_history_enabled', true) }}" 

(Notice the condition is now fully inside the Jinja, rather than templating a string)__

Relevant error log or model output

No response

Expected behavior

Does not raise validation error

dbt Project configurations

vars:
  ...
  # Ads Reporting
  ad_reporting__amazon_ads_enabled: False

Package versions

17:42:18  Installing fivetran/amazon_ads_source
17:42:20    Installed from version 0.1.0
17:42:20    Up to date!

What database are you using dbt with?

snowflake

dbt Version

$ dbt --version
Core:
  - installed: 1.4.0
  - latest:    1.4.1 - Update available!

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]

Thanks again for raising this to our team and helping to address this issue! This has since been addressed within PR #5 and is now live on the hub.

As such, the issue should be addressed and I will close this bug report. Please feel free to reopen if the issue persists on your end 😄