-
Notifications
You must be signed in to change notification settings - Fork 12
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
F/facebook insights #74
Conversation
7f5baa8
to
c3fa6f8
Compare
Codecov Report
@@ Coverage Diff @@
## master #74 +/- ##
==========================================
+ Coverage 97.69% 97.75% +0.06%
==========================================
Files 27 28 +1
Lines 1084 1115 +31
==========================================
+ Hits 1059 1090 +31
Misses 25 25
Continue to review full report at Codecov.
|
doc/facebook_insights.md
Outdated
* `password`: str | ||
* `port`: int | ||
* `charset`: str, default to utf8mb4 | ||
* `connect_timeout`: int |
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.
I think these parameters come from a copy paste and does not relate to Facebook Insights ;)
doc/facebook_insights.md
Outdated
|
||
```javascript | ||
DATA_PROVIDERS: [ | ||
type: 'faceobok_insights', |
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.
'facebook_insights'
doc/facebook_insights.md
Outdated
## Data source configuration | ||
|
||
|
||
Either `query` or `table` are required, both at the same time are not supported. |
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.
I think this comment comes from a copy paste, and is not related to Facebook Insights ;)
- `name`: the metric name (e.g. `page_total_actions`) | ||
- `period`: the insight collection period (e.g. `week`) | ||
- `title`: a title for the metric returned | ||
- `description`: a description for the metric returnedd |
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.
returned
c3fa6f8
to
6bb91db
Compare
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.
LGTM
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.
LGTM ! Thanks 👍
Translation of a `replace` step in Mongo aggregation pipeline steps. The `replace` step aims to replace value in a column, and write the resulting column inplace or in a new column. Closes [issue #61](ToucanToco/weaverbird#61)
Provide a facbook insights connector.
Check https://developers.facebook.com/docs/graph-api/reference/v2.8/insights_result for the official API documentation.
The connector assumes that you already have a list of page tokens. You can fetch page tokens programmatically but you need a user shortlived token beforehands anyway. For more information about tokens, check https://developers.facebook.com/docs/facebook-login/access-tokens/
Here's a sample python script to fetch all page tokens available to a given user