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.

Sign up for GitHub

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: Allow configuring insights lookback window #3396

Merged
merged 9 commits into from
May 20, 2021
Merged

🎉 Source Facebook Marketing: Allow configuring insights lookback window #3396

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
56f9c61
added new field
po3na4skld May 13, 2021
6783dcf
lookback window field added to client
po3na4skld May 14, 2021
54e8634
changes after gradlew
po3na4skld May 14, 2021
b8e782c
gradlew changes
po3na4skld May 18, 2021
8de7dee
gradlew changes
po3na4skld May 19, 2021
7b39c0f
Merge branch 'master' of github.com:airbytehq/airbyte into 2920-Allow…
po3na4skld May 19, 2021
ede53e9
Merge branch 'master' into 2920-Allow-configuring-Facebook-Marketing-…
po3na4skld May 19, 2021
a3c70cd
version bump
po3na4skld May 20, 2021
6ce6871
Merge remote-tracking branch 'origin/2920-Allow-configuring-Facebook-…
po3na4skld May 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions ...yte-integrations/connectors/source-facebook-marketing/source_facebook_marketing/spec.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,14 @@
"type": "boolean",
"description": "Include data from deleted campaigns, ads, and adsets.",
"default": "false"
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the first step, but this needs to be passed into the connector so it would actually use this value.

Also, could you add a test using a mocked HTTP request to verify this works?

"insights_lookback_window": {
"type": "integer",
"title": "Insights Lookback Window",
"description": "The attribution window for the actions",
"minimum": 0,
"maximum": 28,
"default": 28
}
}
}
Expand Down