Skip to content
This repository was archived by the owner on Mar 7, 2025. It is now read-only.

Bump exoplayer to 2.13.3#16

Merged
mchowning merged 1 commit intowp-fork-upstream-5.2.0from
wp-fork-upstream-5.2.0+exoplayer-2.13.3
Feb 2, 2022
Merged

Bump exoplayer to 2.13.3#16
mchowning merged 1 commit intowp-fork-upstream-5.2.0from
wp-fork-upstream-5.2.0+exoplayer-2.13.3

Conversation

@mchowning
Copy link
Copy Markdown

@mchowning mchowning commented Feb 1, 2022

This version is available from the Google repository, so it avoids us needing to host it ourselves since jcenter is shutting down soon and is already pretty unreliable.

These same changes are unreleased in the main react-native-video repository: https://github.com/react-native-video/react-native-video/pull/2552/files

I'm targeting your PR with this PR so you can easily see these changes in isolation. That means we'll need to merge both of these before we can update and merge the Gutenberg PR

This version is available from the Google repository, so it avoids us
needing to host it ourselves since jcenter is shutting down soon and
already pretty unreliable
Copy link
Copy Markdown

@hypest hypest left a comment

Choose a reason for hiding this comment

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

LGTM!
I gave this a try on WPAndroid (version alpha-339, against the gb-mobile PR) and the video plays fine as the background of a cover block 👍

@mchowning mchowning merged commit 424c45d into wp-fork-upstream-5.2.0 Feb 2, 2022
@mchowning mchowning deleted the wp-fork-upstream-5.2.0+exoplayer-2.13.3 branch February 2, 2022 16:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants