Skip to content

Commit

Permalink
fix: Exclude deprecated insights fields for v19.0 and above (#262)
Browse files Browse the repository at this point in the history
* Exclude deprecated insights fields

* [pre-commit.ci] auto fixes

* Bump default API version to v19

---------

Co-authored-by: Spencer Beckwith <spencerjbeckwith@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
3 people authored Nov 13, 2024
1 parent d738f7c commit d85b963
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions tap_facebook/streams/ad_insights.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,15 @@
"__module__",
"__doc__",
"__dict__",
# No longer available >= v19.0: https://developers.facebook.com/docs/marketing-api/marketing-api-changelog/version19.0/
"age_targeting",
"gender_targeting",
"labels",
"location",
"estimated_ad_recall_rate_lower_bound",
"estimated_ad_recall_rate_upper_bound",
"estimated_ad_recallers_lower_bound",
"estimated_ad_recallers_upper_bound",
]

SLEEP_TIME_INCREMENT = 5
Expand Down
2 changes: 1 addition & 1 deletion tap_facebook/tap.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class TapFacebook(Tap):
"api_version",
th.StringType,
description="The API version to request data from.",
default="v18.0",
default="v19.0",
),
th.Property(
"account_id",
Expand Down

0 comments on commit d85b963

Please sign in to comment.