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

Video Optimization: VideoPress / Jetpack Integration #3045

Open
swissspidy opened this issue Aug 19, 2019 · 4 comments

Comments

@swissspidy
Copy link
Collaborator

@swissspidy swissspidy commented Aug 19, 2019

As discussed in #2984, we are going with a VideoPress integration in a first step, and will build on top of such an integration to support other solutions as well.

#2984 (comment) mentions a recommended approach, although I am not sure how much of that applies to VideoPress; given that its implementation seems to already work automatically behind the scenes.

However, there currently seems to be an issue in Jetpack that prevents videos uploaded via the video block from being optimized (see Automattic/jetpack#11194, https://jetpack.com/support/jetpack-blocks/video-block/, and https://jetpack.com/support/video-hosting/).

@spacedmonkey

This comment has been minimized.

Copy link
Collaborator

@spacedmonkey spacedmonkey commented Sep 20, 2019

To start work on this, the is required.

  • A jetpack account with a paid for subscription that supports videopress.
  • Add Jetpack plugin to testing environment, activate it and setup VideoPress.
  • Find someway to use videopress locally in a development environment.

I believe this issue is blocked by this Jetpack upstream PR Automattic/jetpack#13395 which doesn't seem to be moving. Until Jetpack is fixed, I recommend moving this out of sprint 14 / 15 and move it to the backlog. Thoughts @MackenzieHartung ?

@spacedmonkey

This comment has been minimized.

Copy link
Collaborator

@spacedmonkey spacedmonkey commented Sep 23, 2019

Interesting and related is this news - https://rtmedia.io/blog/transcoder-for-amp-video-stories

Not heard of rtmedia personally before this was flagged in the slack channel. Plans for this service start from free to $9 a month, which seems pretty reasonable. Plugin already integrated with AMP plugn 1.3+.

@MackenzieHartung MackenzieHartung moved this from To Do to Backlog in AMP Stories Sep 24, 2019
@MackenzieHartung

This comment has been minimized.

Copy link

@MackenzieHartung MackenzieHartung commented Sep 24, 2019

On hold given the rtmedia transcoder option.

@MackenzieHartung MackenzieHartung added this to Prioritization in Definition [Stories] via automation Oct 5, 2019
@MackenzieHartung MackenzieHartung moved this from Prioritization to Estimate in Definition [Stories] Oct 5, 2019
@MackenzieHartung MackenzieHartung moved this from Estimate to Revisit later in Definition [Stories] Oct 5, 2019
@MackenzieHartung MackenzieHartung removed this from Backlog in AMP Stories Oct 7, 2019
@swissspidy

This comment has been minimized.

Copy link
Collaborator Author

@swissspidy swissspidy commented Oct 14, 2019

Some good news: Automattic/jetpack#13682 has been merged, which means Jetpack 7.9 will have better VideoPress integration 💪 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3 participants
You can’t perform that action at this time.