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

[Project. Status] Version 6.0.0-alpha #2723

Closed
hueniverse opened this issue Jun 15, 2022 · 38 comments
Closed

[Project. Status] Version 6.0.0-alpha #2723

hueniverse opened this issue Jun 15, 2022 · 38 comments

Comments

@hueniverse
Copy link
Contributor

hueniverse commented Jun 15, 2022

After a long period of inactivity, developments have restarted in this repository. V6.x branch already contains number of alpha releases.

These releases includes many bug fixes and improvements from many contributors. I want to express my deep gratitude for all the help in getting this important milestone. Many contributors had to rebase their contributions multiple times as some where a few years old.

Please test the new release with your app and report back any issues. Please clearly label your issues as v6 so that we can prioritize them.

Main changes in V6 branch:

  • Remove support for DOM
  • Remove support for Android media player (only support Exo player)
  • Rewrite AVPlayer (ios) binding in swift
  • Add client side ads insertion on ios and android (google ima)
  • Update exoplayer to media3

This branch should now be be stable, we reach the beta state.
Please report any regressions comparing to 5.2.1, they will be handled as a priorityand tagged https://github.com/react-native-video/react-native-video/labels/critical%20for%20release
If you want to contribute, following ticket are a good first approach: https://github.com/react-native-video/react-native-video/labels/good%20first%20issue

Roadmap to be discussed here: #3351
Project status discussions here: #3350

(Edited by @freeboub)

@hueniverse hueniverse pinned this issue Jun 15, 2022
@JstnEdr
Copy link

JstnEdr commented Jun 16, 2022

Looks like fullscreen video on Android is still not supported. Are there any plans to address this in v6?

@hueniverse
Copy link
Contributor Author

@JstnEdr if there is a PR ready or someone willing to take the lead, sure. There are no free resources around atm...

@baveku
Copy link

baveku commented Jun 21, 2022

I hope we can custom control UI, on iOS it's beautiful but on Android, we don't have a fullscreen button.
With 0.69.0 we need to integrate using New Architecture, I think we need to migrate this library.

@JongGyuChoi
Copy link

A build error occurred in RCTVideoManager
Screen Shot 2022-06-22 at 10 57 01 AM
Screen Shot 2022-06-22 at 10 57 07 AM

.

@baveku
Copy link

baveku commented Jun 22, 2022

A build error occurred in RCTVideoManager Screen Shot 2022-06-22 at 10 57 01 AM Screen Shot 2022-06-22 at 10 57 07 AM

.

I guess you use version v6 on npm,
so you need to move from npm to github repo
"react-native-video": "react-native-video/react-native-video"
to fix it

@hueniverse hueniverse changed the title Version 6.0.0-alpha.0 Version 6.0.0-alpha Jun 22, 2022
@Dr-Horv
Copy link

Dr-Horv commented Jun 27, 2022

@hueniverse Any ETA how long the alpha vill be ongoing and when the 6.0 release will happen?

Not asking to stress, but need to know for a project I'm working on if we need to go through the hoops (documentation, signoff etc) of using an alpha-version of a library in our next production release since we have conflicts with another dependency on the Exoplayer and can't keep using 5.2

@hueniverse
Copy link
Contributor Author

@Dr-Horv Hard to say but hopefully an official v6 by end of July. The main blocker is that we only have 2-3 people actually doing the work. If more people reviewed PRs and approved them (or report issues), we would be moving much faster.

@rafaelbrun
Copy link

@hueniverse @freeboub or anyone that can provide some information. Is there any updates on an ETA for the 6.0 release? And what is needed so that this release can happen?

I'm working in a project that our main resource is using this lib, me and my team may be able to help if we have some information about how much work is left to get this release out.

You can reach me out on my email if you think that's better: [email protected]

Thanks for you hard work on this amazing lib!!

@hueniverse
Copy link
Contributor Author

@rafaelbrun see: #2788

@rafaelbrun
Copy link

@hueniverse @freeboub or anyone that can provide some information. Is there any updates on an ETA for the 6.0 release? And what is needed so that this release can happen?

I'm working in a project that our main resource is using this lib, me and my team may be able to help if we have some information about how much work is left to get this release out.

You can reach me out on my email if you think that's better: [email protected]

Thanks for you hard work on this amazing lib!!

@cobarx

@freeboub
Copy link
Collaborator

@rafaelbrun
Hello, I invite you on a slack channel.
I advice you to fork this repo for your corporate usage...
I will take the lead for releases, but I am doing it only on my spare time, so I cannot be 'reactive' every days :)
Thank you.

@smikheiev
Copy link

are there some plans about releasing 6.0.0-alpha.2? there's 6.0.0-alpha.2 and even 6.0.0-alpha3 in the changelog, but the latest released version in npm is 6.0.0-alpha1? 🤔

@freeboub
Copy link
Collaborator

Version 6.0.0.alpha.3 has been released on npm

@jemise111
Copy link
Contributor

Hey @hueniverse @freeboub (or anyone else who can help). Thank you for all of the hard work on v6. We're trialing using the latest alpha in our app, as it is required for compatibility with another dependency we're using, react-native-track-player.

I was wondering if there is a list of known issues/risks in the latest alpha release anywhere, so we may know what to look out for in testing the stability of v6 in its current state?

Thanks for any help 🙏

@Lakston
Copy link

Lakston commented Oct 25, 2022

In alpha 3 you are using a fork of ScalableVideoView, is that really needed ?

It is causing Duplicate class issues on android builds if another lib is using ScalableVideoView.

@freeboub
Copy link
Collaborator

@Lakston ScalableVideoView is not used anymore in V6.0.0
@jemise111 there is no possibility to use mulitple exoplayer version in my knowledge ...

@HaddyYang
Copy link

The fullscreenOrientation attribute is not work in [email protected], both Android and iOS.
My react-native version is 0.70.3

@DennisdeWitNL
Copy link

DennisdeWitNL commented Nov 1, 2022

alpha.3 completely breaks iOS

node_modules/react-native-video/ios/Video/RCTVideo.swift:24:34: error build: Cannot find type 'RCTEventDispatcher' in scope

@zchwyng
Copy link
Contributor

zchwyng commented Nov 21, 2022

When will alpha 4 be released to npm?

@marcel-happyfloat
Copy link
Contributor

marcel-happyfloat commented Dec 3, 2022

alpha 3 works great for me in development, but in release build it keeps crashing almost instantly (most of the time, but sometimes it takes a while of scrolling up and down in my item feed) with this error:

EXC_BREAKPOINT react_native_video.RCTVideoPlayerViewController
reactViewController > superview > ticalCornerRadius:

Similar issues grouped together in sentry error reporting, e.g:

RNSScreen | EXC_BREAKPOINT: dyld_v1 arm64e > iOntology27UsoEntity_common_Arithmetic > reactViewController > > superview

EXC_BREAKPOINT react_native_video.RCTVideoPlayerViewController
ePower > reactViewController > superview

Anybody has an idea whats wrong?

alpha 4 is not yet released to test.

Edit: I tried to set a fixed video height to the parent container too, but I don't think it did change anything.
Edit: I also added this patch to my alpha 3 version from alpha 4: #2907 and I think this issue is new, but not sure.

NSInternalInconsistencyException react_native_video.RCTVideoPlayerViewController
NSInternalInconsistencyException: Cannot remove an observer <NSKeyValueObservance 0x28266b390> for the key path "currentItem.playbackBufferEmpty" from <AVPlayer 0x282ae9970>, most likely because the value for the key "currentItem" has changed without an appropriate KVO notification being sent. Check the KVO-compliance of the AVPlayer class.

@marcel-happyfloat
Copy link
Contributor

This time I catched some error in XCode and I could solve my problem by doing this:

diff --git a/node_modules/react-native-video/ios/Video/RCTVideo.swift b/node_modules/react-native-video/ios/Video/RCTVideo.swift
index 47231a7..3dc48e1 100644
--- a/node_modules/react-native-video/ios/Video/RCTVideo.swift
+++ b/node_modules/react-native-video/ios/Video/RCTVideo.swift
@@ -646,7 +646,7 @@ class RCTVideo: UIView, RCTVideoPlayerViewControllerDelegate, RCTPlayerObserverH
         
         if _controls {
             let viewController:UIViewController! = self.reactViewController()
-            viewController.addChild(_playerViewController)
+            viewController?.addChild(_playerViewController)
             self.addSubview(_playerViewController.view)
         }

@freeboub
Copy link
Collaborator

freeboub commented Dec 6, 2022

@happyfloat can you please open a PR with this change ?

marcel-happyfloat added a commit to marcel-happyfloat/react-native-video that referenced this issue Dec 7, 2022
@tsalama
Copy link

tsalama commented Feb 21, 2023

Hey guys, just curious if there's a list of outstanding TODOs and necessary tasks before v6 officially goes live? I see alpha.5 was released last week though, awesome work.

@Larens94
Copy link

@Lakston ScalableVideoView is not used anymore in V6.0.0 @jemise111 there is no possibility to use mulitple exoplayer version in my knowledge ...

how did you solve the problem?
I currently receive:
Caused by: java.lang.RuntimeException: Duplicate class com.yqritc.scalablevideoview.BuildConfig found in modules jetified-Android-ScalableVideoView-10e7b4da8b-runtime (com.github.adityaxjha:Android-ScalableVideoView:10e7b4da8b) and jetified-android-scalablevideoview- v1.0.4-jitpack-runtime (com.github.MatrixFrog:android-scalablevideoview:v1.0.4-jitpack)

@eugened-ith
Copy link

Hi! Coming from expo-av because of android crash. Tried both react-native-video v5.2.1 and
v6.0.0-alpha.6 the app crashes with alpha with same error as expo-av with the next stack trace

java.lang.OutOfMemoryError: Failed to allocate a 2944 byte allocation with 47976 free bytes and 46KB until OOM, target footprint 268435456, growth limit 268435456; giving up on allocation because <1% of heap free after GC.
    at java.lang.AbstractStringBuilder.<init>(AbstractStringBuilder.java:69)
    at java.lang.StringBuilder.<init>(StringBuilder.java:102)
    at java.lang.String.replace(String.java:2787)
    at com.google.android.exoplayer2.util.Log.appendThrowableString(Log.java:295)
    at com.google.android.exoplayer2.util.Log.e(Log.java:254)
    at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:440)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
    at java.lang.Thread.run(Thread.java:1012)

The context is that I'm working on a twitter-like app and have a flash list of videos and other media that comes from IPFS and they can have rather big size, probably up to 1.5GB but I don't have any ability to compress these files anyhow so they come to app as is.

I'll go with v5.2.1 now as it just works now, but I'm wondering what can cause such problem in both react-native-video and expo-av.

I also have enabled android:largeHeap and android:hardwareAccelerated.

Thank you!

@lutfullo777
Copy link

Is getting videoTrack added for IOS when onLoad video?

@tsalama
Copy link

tsalama commented Aug 30, 2023

Hi all, just curious if there's a list of outstanding TODOs and necessary tasks blocking the official v6 release? I see alpha.7 was released last month though, awesome work to those who contributed.

@freeboub
Copy link
Collaborator

@tsalama the list of ticketed I tagged as mandatory:
https://github.com/react-native-video/react-native-video/issues?q=is%3Aopen+is%3Aissue+label%3A%22critical+for+release%22

I hope it will be enough to have at least the same stability than 5.2

@tsalama
Copy link

tsalama commented Aug 31, 2023

@freeboub Ok great to know thank you.

@bulkinav
Copy link

bulkinav commented Oct 5, 2023

Currently the 'pictureinpicture' pros only support IOS. Is there an understanding when it will be available for Android?

I see the old PR for react-native-video 5.x (#1776), but would like to see and for 6.x

In my opinion, this is a very popular feature.

@freeboub freeboub changed the title Version 6.0.0-alpha [Project. Status] Version 6.0.0-alpha Oct 5, 2023
@boiboif
Copy link

boiboif commented Nov 1, 2023

Hello, when will alpha 9 be released to npm? I need typescript support. Thanks.

@vargajacint
Copy link

@boiboif You can add "@types/react-native-video" to your devDependencies until it's not released, or you can specify a remote branch name instead of the version number

@boiboif
Copy link

boiboif commented Nov 2, 2023

@boiboif You can add "@types/react-native-video" to your devDependencies until it's not released, or you can specify a remote branch name instead of the version number

@vargajacint Thanks!

@freeboub
Copy link
Collaborator

freeboub commented Nov 4, 2023

Dear all,
alpha.9 has finally be released ! 🚀
Notice that the typescript support has finally be integrated.
Here is change log: https://github.com/react-native-video/react-native-video/releases
Thank you to all contributors, and special thank to @KrzysztofMoch !

@EvelynGuoEntain
Copy link

Hi team, when will the migration to androix.media3 be included in new release? Thanks.

@avencat
Copy link
Contributor

avencat commented Nov 22, 2023

Hi @EvelynGuoEntain it seems it is built in 6.0.0-beta.0

@afkcodes
Copy link

Hello is MergingMediaSource is going to be available in 6.0 or is it planned to be supported ?

@freeboub
Copy link
Collaborator

let's continue this discussion in #3350

@KrzysztofMoch KrzysztofMoch unpinned this issue May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests