-
Notifications
You must be signed in to change notification settings - Fork 31
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: Webhooks for VOD #1099
api: Webhooks for VOD #1099
Conversation
Create and initial status update on enqueue for now
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report
@@ Coverage Diff @@
## master #1099 +/- ##
===================================================
+ Coverage 50.21750% 50.42077% +0.20326%
===================================================
Files 66 66
Lines 4138 4159 +21
Branches 730 733 +3
===================================================
+ Hits 2078 2097 +19
- Misses 1814 1815 +1
- Partials 246 247 +1
Continue to review full report at Codecov.
|
PR description that Github is not showing: |
Aha! The PR template apparently came with a big comment that wasn't closed, then everything was hidden lol Fixed now |
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.
Webhook logic LGTM, just some nits
What does this pull request do? Explain your changes. (required)
This implements a couple of webhooks for the VOD feature in general, as
per this short discussion/spec.
Specific updates (required)
-
yarn test
(might write some explicit tests for these, idk)Does this pull request close any open issues?
Implements #1005
Checklist: