-
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: Propagate progress field to asset #1244
Conversation
Also add processing phase to assets so one doesn't have to use the existence of the progress field to tell if it has started or not.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report
@@ Coverage Diff @@
## master #1244 +/- ##
===================================================
- Coverage 51.37552% 51.29439% -0.08114%
===================================================
Files 66 66
Lines 4362 4365 +3
Branches 791 794 +3
===================================================
- Hits 2241 2239 -2
- Misses 1838 1842 +4
- Partials 283 284 +1
Continue to review full report at Codecov.
|
Stop merging hell. Only the retries field had to be copied around over and over, which is better than clearing the progress and step anyway.
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.
With this pull request, I am unable to open the asset page on the dashboard in dev
. It seems to be related to the views
api failing tho. Apart of this, LGTM
0727a4d
to
2d9f567
Compare
What does this pull request do? Explain your changes. (required)
Also add processing phase to assets so one doesn't have
to use the existence of the progress field to tell if it
has started or not.
Specific updates (required)
progress
field to assetsprocessing
phase to assets-
yarn test
Does this pull request close any open issues?
Unofficial issue discussed yesterday for SDK to unblock livepeer/ui-kit#27
Checklist: