Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

most of streams in Facebook marketing not working good | update Facebook marketing API field to be appropriate with API 19 #103

Closed
eslamaboushashaa opened this issue Mar 5, 2024 · 4 comments
Assignees

Comments

@eslamaboushashaa
Copy link

eslamaboushashaa commented Mar 5, 2024

Note from @aaronsteers (maintainer):

Duplicate of:

Pull request pending:


Original issue description:

I received an error message as some fields that I was trying to access do not exist in the Facebook API V-19.
I got these fields

  • 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

Full error
Error: 100, (#100) 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 are not valid for fields param after V19.0.

@aaronsteers
Copy link
Contributor

aaronsteers commented Mar 5, 2024

@eslamaboushashaa - Thanks very much for reporting this issue. As a first step, we will see if I can reproduce this error.

Do you have a PyAirbyte code sample that can share with us? It is helpful if you include the "config" entry - removing or redacting any secrets, of course.

Also - were you able to find any workaround as of yet, or is this a blocker for you?

Thanks again!


Update:

Attaching here a notice of these fields deprecated as of Marketing API v0.19.0, dated Jan 23, 2024.

@aaronsteers
Copy link
Contributor

@aaronsteers
Copy link
Contributor

aaronsteers commented Mar 5, 2024

I found an open PR here:

Was approved yestersday, expected to merge and release soon.

@aaronsteers
Copy link
Contributor

Closing (dedupe), in favor of:

as primary.

Please upvote that issue to get updates, and/or re-open if this is still needed to be separately tracked.

Thanks again for logging. Hopefully will close out with successful resolution very soon.

@aaronsteers aaronsteers closed this as not planned Won't fix, can't repro, duplicate, stale Mar 5, 2024
@aaronsteers aaronsteers self-assigned this Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants