You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/10/29 12:05:20 UTC

[GitHub] [airflow] bugraoz93 commented on issue #19269: Add support for multiple AdAccount when getting Bulk Facebook Report from Facebook Ads Hook

bugraoz93 commented on issue #19269:
URL: https://github.com/apache/airflow/issues/19269#issuecomment-954688161


   Hey @mik-laj, @randr97, and @eladkal, thank you for all of your quick responses and assistance. 
   @randr97 That is one of my concerns too. I agree with your idea about serial reporting. I will do it like that. If it is okay for you too, I am also thinking of updating the default API version to 11.0 since I will do some changes on the hook. It could help people who forget to put the API version to the operator. Otherwise, Facebook Business API is throwing an error to use 11.0 instead.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org