bugraoz93 commented on a change in pull request #19377:
URL: https://github.com/apache/airflow/pull/19377#discussion_r747630615
##########
File path: airflow/providers/facebook/ads/hooks/ads.py
##########
@@ -98,14 +99,17 @@ def facebook_ads_config(self) -> Dict:
if missing_keys:
message = f"{missing_keys} fields are missing"
raise AirflowException(message)
+ # Solve a way to check in bulk_facebook_report to handle list and dict
return there
+ if type(config["account_id"]) is not list:
Review comment:
Normally, the `account_id` was given as a string in the Facebook Ads
Connection in the Airflow, with this check it is controlling rather the
connection is the old version or is a new version which is an array of
`account_id`. Also, I have updated this part regarding your suggestion.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
