-
Notifications
You must be signed in to change notification settings - Fork 5
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
api/views: Create new viewership BigQuery API #121
Conversation
197e4c5
to
2bade05
Compare
4cbf3f2
to
b2a54e7
Compare
b2a54e7
to
7d64047
Compare
Codecov Report
@@ Coverage Diff @@
## main #121 +/- ##
===============================================
+ Coverage 0 25.63291% +25.63291%
===============================================
Files 0 4 +4
Lines 0 316 +316
===============================================
+ Hits 0 81 +81
- Misses 0 223 +223
- Partials 0 12 +12
Continue to review full report in Codecov by Sentry.
|
b11d705
to
799ea75
Compare
Also fix exits_before_start tag
0a27eff
to
2241bd8
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 with comments!
64ee9f0
to
99c027c
Compare
Re-ordered the shit out of bigquery handler
99c027c
to
3d23fdd
Compare
This implements the new Viewership V2 API in the analyzer service. It uses bigquery as the
data store and implements all the features proposed for the April project.
This implements DAT-62.