-
Notifications
You must be signed in to change notification settings - Fork 147
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
Added Facebook Broadcasts API coverage #174
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #174 +/- ##
=========================================
Coverage 15.78% 15.78%
Complexity 190 190
=========================================
Files 23 23
Lines 646 646
=========================================
Hits 102 102
Misses 544 544
Continue to review full report at Codecov.
|
Oops I didn't see the tests coming. I think tests should be moved to a different namespace, right? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #174 +/- ##
=========================================
Coverage 15.78% 15.78%
Complexity 190 190
=========================================
Files 23 23
Lines 646 646
=========================================
Hits 102 102
Misses 544 544 ☔ View full report in Codecov by Sentry. |
Hi! I just added coverage for Facebook Broadcasts API (sending messages to groups of users) and labeling, according to documentation.
Broadcast reach evaluation and some minor features were not implemented.
Tests with complete coverage for the new features are included in PR.
MAIN DOC
TARGETING BROADCASTS DOC