video-transcoding-api icon indicating copy to clipboard operation
video-transcoding-api copied to clipboard

Published 20 hours ago • verified publisher icon video-dev

provider: create helpers module

Open flavioribeiro opened this issue 7 years ago • 1 comments

I could find some similar code on encodingcom and zencoder integration. I bet we can find more. We should create a helpers module to avoid duplicated code:

Examples: https://github.com/NYTimes/video-transcoding-api/blob/master/provider/encodingcom/encodingcom.go#L130-L138 and https://github.com/NYTimes/video-transcoding-api/blob/master/provider/zencoder/zencoder.go#L189-L200

https://github.com/NYTimes/video-transcoding-api/blob/master/provider/encodingcom/encodingcom.go#L324-L330 and https://github.com/NYTimes/video-transcoding-api/blob/master/provider/zencoder/zencoder.go#L312-L316

flavioribeiro avatar Nov 23 '16 18:11 flavioribeiro

another example: https://github.com/NYTimes/video-transcoding-api/blob/master/provider/encodingcom/encodingcom.go#L340-L351 and https://github.com/NYTimes/video-transcoding-api/blob/master/provider/zencoder/zencoder.go#L299-L312

flavioribeiro avatar Nov 23 '16 22:11 flavioribeiro