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
Source Facebook Marketing: Add Ad Insights streams #26720
Source Facebook Marketing: Add Ad Insights streams #26720
Conversation
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
/test connector=connectors/source-facebook-marketing
Build FailedTest summary info:
|
/test connector=connectors/source-facebook-marketing
Build FailedTest summary info:
|
/test connector=connectors/source-facebook-marketing
|
…ights-streams-17238' into artem1205/source-facebook-ad-insights-streams-17238
/test connector=connectors/source-facebook-marketing
Build PassedTest summary info:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
@@ -27,14 +27,44 @@ acceptance_tests: | |||
bypass_reason: "Thumbnail urls changes permanently" | |||
- name: "ad_creatives" | |||
bypass_reason: "Thumbnail urls changes permanently" | |||
- name: "ads_insights_action_carousel_card" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how are we testing the new streams if there's no data in our test account?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do have some data for all these streams. Or, more precisely, all ad Insight streams make requests to the same data, but using different "slicing" columns (like in pivot tables).
I've disable them for basic_read
test, because we use our "live" account for testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We still test all these streams in incremental
and full_refresh
tests
* Source Facebook Marketing: Add Ad Insights streams * Source Facebook Marketing: update docs; bump version * Source Facebook Marketing: update tests config * Source Facebook Marketing: fix unit test * Source Facebook Marketing: typo fix * Source Facebook Marketing: update future_state * Source Facebook Marketing: fix expected records
What
Resolve #17238
Add Custom Ad Insights streams
How
Add new Reports using custom Ad Insights
breakdowns
andaction_breakdowns
combinations.See Docs update for more info.
Recommended reading order
y.python
🚨 User Impact 🚨
No Breaking Changes
Pre-merge Actions
Updating a connector
Community member or Airbyter
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.