Skip to content
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

Add HTTP Ingestion Appliance #37

Merged
merged 1 commit into from
Aug 26, 2020
Merged

Conversation

slifty
Copy link
Member

@slifty slifty commented Aug 23, 2020

Description

This PR adds an appliance that can convert a video URL into STREAM.CONTAINER payloads.

Due Diligence Checklist

  • Tests
  • Documentation
  • Consolidated commits
  • Relevant labels assigned
  • Changelog(s) updated

Steps to Test

  1. yarn test

Deploy Notes

  • yarn install

Related Issues

This is the final ingestion engine to be converted, so it resolves #32
Relies on #36

@slifty slifty force-pushed the 32-add-http-ingestion-appliance branch 2 times, most recently from 0b1f476 to 9669521 Compare August 25, 2020 19:28
Copy link
Contributor

@chriszs chriszs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having written at least some of this, I approve of it.

The @tvkitchen/appliance-video-http-ingestion package will take a given
video URL and convert it to STREAM.CONTAINER payloads.

Issue #32

Co-authored-by: Chris Zubak-Skees <chriszs@gmail.com>
@slifty slifty force-pushed the 32-add-http-ingestion-appliance branch from 9606b55 to 6c2c95c Compare August 26, 2020 17:31
@slifty slifty merged commit 4cff0d7 into master Aug 26, 2020
@slifty slifty deleted the 32-add-http-ingestion-appliance branch August 26, 2020 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create ingestion engine appliances(s)
2 participants