potiuk commented on a change in pull request #12558:
URL: https://github.com/apache/airflow/pull/12558#discussion_r537046911
##########
File path: airflow/providers/facebook/ads/hooks/ads.py
##########
@@ -55,9 +55,15 @@ class FacebookAdsReportingHook(BaseHook):
"""
+ conn_name_attr = 'facebook_conn_id'
+ default_conn_name = 'facebook_default'
+ conn_type = 'facebook_social'
+ hook_name = 'Facebook Social'
Review comment:
Good question. I took it from "extra-packages" reference. But when you
look at the API it is actually called 'Facebook Marketing API" - but indeed
this one uses the "Ads" part. I will replace it here but leave `Facebook
Social` in "extras" as there might be more operators added in the future - both
marketing and non-marketing ones.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
